[PATCH] D21005: Add a -verify-after-all option

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 5 15:22:39 PDT 2016


davide added a comment.

In http://reviews.llvm.org/D21005#449357, @sanjoy wrote:

> In http://reviews.llvm.org/D21005#449348, @davide wrote:
>
> > Oh, also, do you have any (easy way) to add a test for this option?
>
>
> I can add a test that makes sure the function exists and is supported by opt (do you want me to do that?), but I don't see how I can test for a bad transform without adding a deliberately broken pass (which seems overkill).  But suggestions are welcome.


Something that tests the basic functionality works is fine. I really don't think adding a broken pass is needed.


http://reviews.llvm.org/D21005





More information about the llvm-commits mailing list