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

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

sorry to be late to reviewing this, but I think this is overly aggressive. I see verifier failures like `scope must have two or three operands` after this patch.

https://reviews.llvm.org/D132077 was in the process of being reviewed and basically does this but more targeted with a specific focus on not creating verifier failures (since that's the direction we'd like to take llvm-reduce, not producing verifier failures since that slows down reduction a lot and dumps a lot of output). could we revert this and use the other patch?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135237