leonardchan added a comment.

In https://reviews.llvm.org/D52814#1262550, @philip.pfaffe wrote:

> You're right, my bad; I missed the fact that it's EP_OptimizerLast, which has 
> no equivalent in the new PM. Your implementation is actually correct here.
>
> Did you run your test in a debug build?


Yes, and I tested it on release also in case there were any differences between 
either builds.


Repository:
  rC Clang

https://reviews.llvm.org/D52814



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

Reply via email to