[PATCH] D49297: expose debugify as an LLVM option in clang

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 15:51:26 PDT 2018


vsk added a comment.

First off, @tyb0807, I really appreciate the work you've been doing on LLVM.

With that said, I think this is trying to solve the wrong problem, and I don't think it makes sense to proceed.

There's already a way to find targeted test cases for debug info bugs with opt (see https://gramanas.github.io/posts/finding-debuginfo-loss/). I don't think it makes sense to land a change this invasive to make that process marginally easier. Especially not when there's a more general way to collect IR from a build system using a simple wrapper (see e.g https://github.com/vedantk/scripts/blob/master/generate-ir-cc).

We're bottlenecked by our ability to *fix* -- not simply to find -- these bugs.


Repository:
  rL LLVM

https://reviews.llvm.org/D49297





More information about the llvm-commits mailing list