[PATCH] D32401: [Devirtualization] insert placement new barrier with -O0

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 23 15:15:57 PDT 2017


rjmccall added a comment.

I continue to be really uncomfortable with the entire design of this optimization, which appears to miscompile code by default, but as long as nobody's suggesting that we actually turn it on by default, I guess it can be your little research-compiler playground.  It might be better to downgrade it to a -cc1 option, though.

This specific change is fine by me.


https://reviews.llvm.org/D32401





More information about the cfe-commits mailing list