[cfe-commits] [PATCH] #import directive in MS compatibility mode

Aaron Ballman aaron at aaronballman.com
Sat Mar 17 17:00:11 PDT 2012


On Thu, Mar 15, 2012 at 7:29 AM, Aaron Ballman <aaron at aaronballman.com> wrote:
> On Wed, Mar 14, 2012 at 11:25 PM, J B <jayblanchard.cpp at gmail.com> wrote:
>> The reason I recommend an error is because the code will error out later
>> without the .tlh (where it is requested via attributes or absence of).  The
>> errors later will be a direct result of the failure of #import to function
>> properly.
>
> That's a valid reason to error in my book.  I'll go down that route.
> Thanks for the insights!

This is the revised patch based on feedback from everyone.  It causes
a (shorter) error to happen when using #import in MS mode, and does
not perform the actual include.  It also eats the optional attributes
so that parsing can continue.

Again, this will fix PR 10727

Thanks!

~Aaron
-------------- next part --------------
A non-text attachment was scrubbed...
Name: MicrosoftImport.patch
Type: application/octet-stream
Size: 4235 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120317/3eb2ed5a/attachment.obj>


More information about the cfe-commits mailing list