[llvm] [DebugInfo][RemoveDIs] Support DPValues in HWAsan (PR #78731)

Jeremy Morse via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 3 09:01:16 PST 2024


================
@@ -578,6 +578,8 @@ bool AArch64StackTagging::runOnFunction(Function &Fn) {
     // Fixup debug intrinsics to point to the new alloca.
     for (auto *DVI : Info.DbgVariableIntrinsics)
       DVI->replaceVariableLocationOp(OldAI, Info.AI);
+    for (auto *DPV : Info.DbgVariableRecords)
----------------
jmorse wrote:

It's not tested under the default configuration and so the lack-of-failure is expected. To ensure the code isn't rotting, we've got this [0] buildbot set up to exercise the "new debug-info" configuration, which requires passing `LLVM_EXPERIMENTAL_DEBUGINFO_ITERATORS=On` to cmake. With that enabled, the test RUN lines where `--try-experimental-debuginfo-iterators` has been added will operate under "new" debug-info mode, and  you Should (TM) see a test failure.

We're inches away from turning this on-by-default and it being tested everywhere, after which there won't be this obscure testing arrangement happening any more.

[0] https://lab.llvm.org/buildbot/#/builders/275

https://github.com/llvm/llvm-project/pull/78731


More information about the llvm-commits mailing list