[PATCH] D106660: [DebugInfo][InstrRef] Don't break up return-sequences on debug-info instructions

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 23 07:50:32 PDT 2021


dblaikie added a comment.

(I wouldn't insist this is held up until a test is added - but I've got a strong preference for a test to be added. If it's hard to find a case, perhaps adding an assertion (faster than a breakpoint/running the whole thing under a debugger) `isDebugValue || !isDebugInstr` I think would be the interesting property to assert - and then hopefully running that over a codebase should yield something, then creduce/llvm-reduce checking for that assertion might reduce out to something useful?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106660



More information about the llvm-commits mailing list