rsmith added a comment.

From a big-picture perspective, I would like us to move to treating MS and GNU 
extensions in the same way (at the cc1 level) to the extent that we can. I 
think this moves us in that direction.

One comment, though: I'm not convinced that the `__EXCEPTIONS` macro should be 
controlled by this flag. Many compilers define this when exceptions are 
available (ARM, IBM, HP, EDG in all modes -- including MSVC-compatible mode, 
...) and it's not obvious to me whether it is in fact a GNU extension or has 
more history than that. (It's also a somewhat-documented 
<http://releases.llvm.org/3.6.0/tools/clang/docs/ReleaseNotes.html#the-exceptions-macro>
 Clang feature independent of GCC compatibility.)

Do you know why we turn that macro off under `MSVCCompat`? If EDG can get away 
with defining it in MSVC-compatible mode, I would expect that we can too.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68055/new/

https://reviews.llvm.org/D68055



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

Reply via email to