[PATCH] D28047: Remove the '-disable-llvm-passes' flag (which I didn't even know existed, and I suspect many others aren't aware of either) and strength '-disable-llvm-optzns' to do the same thing.

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 22 11:28:06 PST 2016


rnk added a comment.

In https://reviews.llvm.org/D28047#629889, @chandlerc wrote:

> I only have a mild preference for how we spell the name based on the fact that i've told people to use -disable-llvm-optzns for years. If we want to go with ...-passes, fine with me.


We have some freedom to change the -cc1 interface. I liked being able to pass -g to -cc1, but then we went to -debug-info-kind=(limited|full|linetables) or whatever. =?

> I'll probably just add an alias so that you can use either spelling, but change the tests to consolidate on whichever spelling we end up with.
> 
> SG? I can regenerate the patch if desired, but it'll just be a lot of perl...

Sounds good! We definitely don't need both -disable-llvm-optzns and -disable-llvm-passes.


https://reviews.llvm.org/D28047





More information about the cfe-commits mailing list