[PATCH] D30430: Make Lit tests C++11 compatible - IR ordering

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 27 21:38:29 PST 2017


rjmccall added a comment.

The C++98 behavior here is not really vital to test precisely; it's just minor differences in what gets instantiated and when.  I think it's fine to just update the run line to -std=c++11 for things like this.  But if you really want to test both configurations, this LGTM, although please leave a comment in the test explaining that it's just trying to account for differences in instantiation order between C++98 and C++11.


https://reviews.llvm.org/D30430





More information about the cfe-commits mailing list