[llvm] [RemoveDIs] Add DPLabels support [3a/3] (PR #82633)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 22 10:18:46 PST 2024


================
@@ -1,5 +1,6 @@
 ; RUN: llc < %s -stop-after=virtregrewriter -o - | FileCheck %s
-;
+; RUN: llc --try-experimental-debuginfo-iterators < %s -stop-after=virtregrewriter -o - | FileCheck %s
----------------
OCHyams wrote:

I mention this in the patch summary - my thinking is that this preserves coverage if we need to disable RemoveDIs temporarily. Roping in @jmorse to see if he's got an opinion on testing strategy going forward.

>I think we'd need one with --experimental-debuginfo-iterators=false

However, I don't think this question (above) is for this patch, as that could arguably be applied to all tests with debug info?


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


More information about the llvm-commits mailing list