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

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

In https://reviews.llvm.org/D49297#1166117, @tyb0807 wrote:

> Hi @vsk,
>
> Thank you for your comment. I agree with what you say, however, I think you may understand that we should have had this discussion before this patch in https://bugs.llvm.org/show_bug.cgi?id=37792 😭


Keep in mind that folks can't respond to everything they're CC'd on. 'Would-be-nice' feature requests like this one necessarily receive less attention. To save time in the future, let people know what you're working on. Assigning a bugzilla PR to yourself, sending out a RFC or an implementation plan on llvm-dev, or even pinging people on IRC are all good ways to do that.

> Anyway, I think we can still reuse part of the logic of this patch to extend the `-verify-each` mode, so that it also is enabled after each module/function pass? What do you think? Do you still need this feature of `-verify-each`?

If you're interested in adding/changing a -verify-each mode in clang/opt, I suggest sending out a RFC. That allows more people to participate in the discussion.


Repository:
  rL LLVM

https://reviews.llvm.org/D49297