I spoke with a Microsoft compiler engineer who said this is most likely a bug that static_assert works in C mode in MSVC. Based on that, I would strongly prefer to not accept this patch.
~Aaron On Wed, Mar 2, 2016 at 9:33 PM, Aaron Ballman <[email protected]> wrote: > I will talk to someone about this tomorrow to see if I can get a > better feel, thank you for the reminder. > > ~Aaron > > On Wed, Mar 2, 2016 at 8:44 PM, Reid Kleckner via cfe-commits > <[email protected]> wrote: >> rnk accepted this revision. >> rnk added a comment. >> This revision is now accepted and ready to land. >> >> lgtm >> >> >> http://reviews.llvm.org/D17444 >> >> >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
