[PATCH] D135237: [llvm-reduce] Remove debug metadata elements

[Paul Robinson via Phabricator via llvm-commits]

probinson added inline comments.


================
Comment at: llvm/test/tools/llvm-reduce/remove-debug-info-nodes.ll:15-18
+; CHECK-INTERESTINGNESS-DAG: !{{DIGlobalVariable}}(
+; CHECK-INTERESTINGNESS-DAG: !{{DILocalVariable}}(name: "A"
+; CHECK-INTERESTINGNESS-DAG: !{{DILocalVariable}}(name: "B"
+; CHECK-INTERESTINGNESS-DAG: !{{DILocalVariable}}(name: "C"
----------------
dblaikie wrote:
> ellis wrote:
> > dblaikie wrote:
> > > The `DILocalVariable`s don't need `{{``}}` do they?
> > Hmm I originally added this because I didn't want these lines to match themselves. But I guess after the first iteration the comments will be removed and they wont' match themselves anymore.
> > 
> > The tests pass without them so I'll remove them.
> oh, fair point - if that's a common stylistic thing done in other llvm-reduce tests, I don't mind it here
It doesn't come up often, but I've seen it done elsewhere.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135237/new/

https://reviews.llvm.org/D135237