[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.

Mehdi AMINI via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 22 09:45:08 PST 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D28047#629746, @rnk wrote:

> How about standardizing on -disable-llvm-passes instead of -disable-llvm-optzns? I never liked "optzns" and can't remember how to spell it. Also, this flag disables LLVM instrumentation passes (ASan) as well as optimization passes, so I think -disable-llvm-passes is the better name.


I agree with that.
To be friendly with developers/users we could print an error mentioning to use `-disable-llvm-passes` when they try to use `-disable-llvm-optzns` (which we accept as a `-mllvm -disable-llvm-optzns` hardcoded in the driver by the way).


https://reviews.llvm.org/D28047





More information about the cfe-commits mailing list