[PATCH] PR9537: Do not warn about unknown pragmas with -Eonly/-M etc.
richard at metafoo.co.uk
Wed Nov 27 16:51:07 PST 2013
Some tiny things:
+ // Ignore also all pragma in all namespaces created
+ // in Preprocessor::RegisterBuiltinPragmas().
The English in this (copy-pasted) comment is a bit awkward; can you fix it
to something like "Also ignore all pragmas in [...]"
Please also add to your testcase an instance of a #pragma we don't normally
understand (#pragma this_pragma_does_not_exist or similar).
Otherwise, LGTM, thanks!
On Wed, Nov 27, 2013 at 3:57 PM, Lubos Lunak <l.lunak at centrum.cz> wrote:
> the attached patch should fix PR9537. It is apparently necessary to
> empty pragma handlers for all pragma namespaces that exist.
> Lubos Lunak
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits