diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2021-05-19 18:55:36 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2021-05-19 18:55:43 +0200 |
commit | 23f7c751268a5cd75494d1ac24b7e585502d8134 (patch) | |
tree | ba882d198509e15a9a6b191f1e79263c33c3fd8c /thirdparty/spirv-reflect/include | |
parent | f1f5b92b8e84541759e7dc38aa85da036027ba06 (diff) |
Android: Remove `-fno-integrated-as`, it can break arm64v8 build
We found that this flag causes this error on PR #48812 which does not add any
fancy inline assembly:
```
/tmp/tile_set-ce236a.s: Assembler messages:
/tmp/tile_set-ce236a.s:34676: Error: selected processor does not support `bfc x0,#32,#32'
clang++: error: assembler command failed with exit code 1 (use -v to see invocation)
```
That flag is mentioned in various errors related to assembler failures on
arm64v8 with Clang from the Android NDK.
It was added in Godot in #6958 when migrating from GCC to Clang, and is indeed
referenced in the NDK's Clang migration guide:
https://android.googlesource.com/platform/ndk/+/master/docs/ClangMigration.md
> Especially for ARM and ARM64, Clang is much stricter about assembler rules
> than GCC/GAS. Use `-fno-integrated-as` if Clang reports errors in inline
> assembly or assembly files that you don't wish to modernize.
We don't get those errors nowadays so it seems the flag is no longer needed.
Diffstat (limited to 'thirdparty/spirv-reflect/include')
0 files changed, 0 insertions, 0 deletions