diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2022-01-25 05:51:14 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-01-25 05:51:14 +0100 |
| commit | 13b87d8cc73b8ac9cab1fa99b2b61820d4d483a6 (patch) | |
| tree | 937d7106779ef49d1997a2178f187af99809a09a /compiler/rustc_codegen_llvm/src/coverageinfo/mod.rs | |
| parent | c3ddca6d4a3cc36337101685f100e0ca9c5e9626 (diff) | |
| parent | e02e9582d253ffaa04c9137f732efb0b80f98907 (diff) | |
| download | rust-13b87d8cc73b8ac9cab1fa99b2b61820d4d483a6.tar.gz rust-13b87d8cc73b8ac9cab1fa99b2b61820d4d483a6.zip | |
Rollup merge of #93269 - jacobbramley:dev/pauth-option-1, r=petrochenkov
Use error-on-mismatch policy for PAuth module flags. This agrees with Clang, and avoids an error when using LTO with mixed C/Rust. LLVM considers different behaviour flags to be a mismatch, even when the flag value itself is the same. This also makes the flag setting explicit for all uses of LLVMRustAddModuleFlag. ---- I believe that this fixes #92885, but have only reproduced it locally on Linux hosts so cannot confirm that it fixes the issue as reported. I have not included a test for this because it is covered by an existing test (`src/test/run-make-fulldeps/cross-lang-lto-clang`). It is not without its problems, though: * The test requires Clang and `--run-clang-based-tests-with=...` to run, and this is not the case on the CI. * Any test I add would have a similar requirement. * With this patch applied, the test gets further, but it still fails (for other reasons). I don't think that affects #92885.
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/coverageinfo/mod.rs')
0 files changed, 0 insertions, 0 deletions
