[PATCH] D42541: [DeadArgumentElimination] Preserve llvm.dbg.values's first argument

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 08:08:36 PST 2018


djtodoro added a comment.

Hi @mattd and @JDevlieghere, thanks a lot for your comments! They sound reasonable, I just updated the patch. I had the idea in my mind about using default bool values, but I saw some solutions for similar problems and made a bunch of handlers, but right now it looks more readable and maintainable. Thanks again!

@aprantl  How does it look now?


https://reviews.llvm.org/D42541





More information about the llvm-commits mailing list