[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 14:48:21 PST 2016


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

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

> In https://reviews.llvm.org/D28047#629892, @mehdi_amini wrote:
>
> > LGTM, but please wait for @rnk  to confirm :)
> >  Also any opinion about a driver option -emit-raw-llvm as @joerg suggested?
>
>
> I like the idea generally, but I don't have an immediate use case and would defer that to a subsequent patch authored by someone who wants this? I really only wanted to remove the divergence between two flags here.


Sure, even @joerg mentioned it was fine to leave for later in his suggestion, I just didn't want it to go unnoticed.


https://reviews.llvm.org/D28047





More information about the cfe-commits mailing list