[cfe-dev] [PATCH] Add Microsoft's handling of exception specifications

Martin Vejnár avakar at ratatanek.cz
Tue Aug 24 03:19:13 PDT 2010


On Tue, 24 Aug 2010 05:51:10 -0400, Francois Pichet <pichet2000 at gmail.com>
wrote:
> I don't think it is necessary to add a MsExceptSpec flag in
> LangOptions nor a -fms-except-spec compiler switch.
> Simply add the following test for MSVC compatibility:
>   if (getLangOptions().Microsoft)
> 
> Because if we add an option or switch for every microsoft extension,
> it is never going to end.

That was my original thought as well, however, the mistreatment of
exception specifications is not strictly speaking an extension. I can
imagine a situation where we would want to support MS extensions and yet
deal with exception specifications correctly.

In any case, I'm attaching another patch that does what you suggest.

Best regards,
-- 
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ms-except-spec-noopt.patch
Type: application/octet-stream
Size: 764 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20100824/09efdbd8/attachment.obj>


More information about the cfe-dev mailing list