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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to