[cfe-commits] [PATCH][MS][Review request] - Late Microsoft template parsing
pichet2000 at gmail.com
Wed Mar 16 06:41:40 PDT 2011
On Tue, Mar 15, 2011 at 2:54 PM, <dawn at burble.org> wrote:
> On Tue, Mar 15, 2011 at 12:29:19AM -0400, Francois Pichet wrote:
>> >>>> I haven't looked at the patch yet, but this defintely needs to be enabled
>> >>>> by its own -cc1 language option; that option should then be enabled by
>> >>>> windows targets, rather than tying this to -fms-extensions.
> Why tie it to windows targets? We would use it on all platforms.
>> >>> Yes. -fdelayed-template-parsing seems a good name.
> Sounds good. Shouldn't that be enough to control the feature, so that
> the target doen't matter?
What I did in the patch is:
+ Opts.DelayedTemplateParsing = Args.hasArg(OPT_fdelayed_template_parsing) ||
The rational is that :
1. Delayed template parsing should always be enabled if
-fms-extensions is present.
2. To give an option to enable delayed template parsing without
3. Don't force delayed template parsing on Windows if
-fms-extensions is not there (some people will want strict template
support on Windows)
PS: virtually all the remaining errors when parsing Microsof C++
template headers with clang are missing "typename". A problem I plan
to fix in later patch.
More information about the cfe-commits