[cfe-commits] r163022 - in /cfe/trunk: include/clang/Lex/Token.h lib/Lex/PPMacroExpansion.cpp lib/Lex/TokenLexer.cpp test/Preprocessor/microsoft-ext.c

[Will Wilson]

Thanks for the info Joåo. Do you think it'll be a lot of work to get to get
it working against the test case?

@Nico: Do you have any idea how prevalent
http://llvm.org/bugs/show_bug.cgi?id=13924 was when r163022 was live? Just
wondering how risky it might be to integrate into my codebase for the time
being.

Thanks!
Will.


On 13 December 2012 01:33, João Matos <ripzonetriton at gmail.com> wrote:

> Actually I needed to apply this patch again yesterday in my local copy,
> and spent an hour debugging the test case Nico added. VS actually supports
> calling macros with less parameters than the ones on the declaration with
> just a warning. Clang will give an error in this case, so I think that's
> related.
>
> I didn't spend much more time on it since for the moment that simple patch
> can get things going in the VS headers.
>
> On Wed, Dec 12, 2012 at 11:40 PM, Will Wilson <will at indefiant.com> wrote:
>
>> Hi João,
>>
>> Re r163022: Emulate MSVC's preprocessor macro argument separator
>> behavior by not considering commas from nested macro expansions as
>> argument separators. Fixes parsing of VS 2012 headers.
>>
>> Did you make any further progress on getting clang compiling against the
>> VS2012 headers? I ask as I'm running into this very issue just now and was
>> rather hoping you might have a solution waiting in the wings ;) If not, I
>> may try and take a look at in in the coming weeks. Any thoughts/patches
>> greatly appreciated!
>>
>> - Will.
>>
>
>
>
> --
> João Matos
>



-- 
*Indefiant Ltd.*
*
*
Firsby Lodge, New Main Road, Scamblesby, Louth, Lincs LN11 9XH UK
*Tel: +44 20 8123 7663 England Registered No. 07936820 VAT No. **128556202*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121213/2c6d57c4/attachment.html>