[cfe-commits] r127910 - in /cfe/trunk: include/clang/Basic/LangOptions.h include/clang/Driver/CC1Options.td lib/Frontend/CompilerInvocation.cpp lib/Lex/Lexer.cpp test/Preprocessor/traditional-cpp.c
Chris Lattner
clattner at apple.com
Fri Mar 18 18:20:18 PDT 2011
On Mar 18, 2011, at 2:23 PM, Daniel Dunbar wrote:
> Author: ddunbar
> Date: Fri Mar 18 16:23:38 2011
> New Revision: 127910
>
> URL: http://llvm.org/viewvc/llvm-project?rev=127910&view=rev
> Log:
> Lexer: Add extremely limited support for -traditional-cpp, ignoring BCPL
> comments.
Hi Daniel,
Why not just have TraditionalCPP turn off LangOpts.BCPLComment?
-Chris
>
> Added:
> cfe/trunk/test/Preprocessor/traditional-cpp.c
> Modified:
> cfe/trunk/include/clang/Basic/LangOptions.h
> cfe/trunk/include/clang/Driver/CC1Options.td
> cfe/trunk/lib/Frontend/CompilerInvocation.cpp
> cfe/trunk/lib/Lex/Lexer.cpp
>
> Modified: cfe/trunk/include/clang/Basic/LangOptions.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/LangOptions.h?rev=127910&r1=127909&r2=127910&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/LangOptions.h (original)
> +++ cfe/trunk/include/clang/Basic/LangOptions.h Fri Mar 18 16:23:38 2011
> @@ -56,6 +56,7 @@
> unsigned ObjCExceptions : 1; // Support Objective-C exceptions.
> unsigned CXXExceptions : 1; // Support C++ exceptions.
> unsigned SjLjExceptions : 1; // Use setjmp-longjump exception handling.
> + unsigned TraditionalCPP : 1; /// Enable some traditional CPP emulation.
> unsigned RTTI : 1; // Support RTTI information.
>
> unsigned MSBitfields : 1; // MS-compatible structure layout
> @@ -169,7 +170,7 @@
> C99 = Microsoft = Borland = CPlusPlus = CPlusPlus0x = 0;
> CXXOperatorNames = PascalStrings = WritableStrings = ConstStrings = 0;
> Exceptions = ObjCExceptions = CXXExceptions = SjLjExceptions = 0;
> - Freestanding = NoBuiltin = 0;
> + TraditionalCPP = Freestanding = NoBuiltin = 0;
> MSBitfields = 0;
> NeXTRuntime = 1;
> RTTI = 1;
>
> Modified: cfe/trunk/include/clang/Driver/CC1Options.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CC1Options.td?rev=127910&r1=127909&r2=127910&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Driver/CC1Options.td (original)
> +++ cfe/trunk/include/clang/Driver/CC1Options.td Fri Mar 18 16:23:38 2011
> @@ -511,6 +511,8 @@
> HelpText<"Store string literals as writable data">;
> def fno_bitfield_type_align : Flag<"-fno-bitfield-type-align">,
> HelpText<"Ignore bit-field types when aligning structures">;
> +def traditional_cpp : Flag<"-traditional-cpp">,
> + HelpText<"Enable some traditional CPP emulation">;
>
> //===----------------------------------------------------------------------===//
> // Header Search Options
>
> Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=127910&r1=127909&r2=127910&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original)
> +++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Fri Mar 18 16:23:38 2011
> @@ -574,6 +574,8 @@
> Res.push_back("-fcxx-exceptions");
> if (Opts.SjLjExceptions)
> Res.push_back("-fsjlj-exceptions");
> + if (Opts.TraditionalCPP)
> + Res.push_back("-traditional-cpp");
> if (!Opts.RTTI)
> Res.push_back("-fno-rtti");
> if (Opts.MSBitfields)
> @@ -1442,6 +1444,7 @@
> Opts.ObjCExceptions = Args.hasArg(OPT_fobjc_exceptions);
> Opts.CXXExceptions = Args.hasArg(OPT_fcxx_exceptions);
> Opts.SjLjExceptions = Args.hasArg(OPT_fsjlj_exceptions);
> + Opts.TraditionalCPP = Args.hasArg(OPT_traditional_cpp);
>
> Opts.RTTI = !Args.hasArg(OPT_fno_rtti);
> Opts.Blocks = Args.hasArg(OPT_fblocks);
>
> Modified: cfe/trunk/lib/Lex/Lexer.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/Lexer.cpp?rev=127910&r1=127909&r2=127910&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Lex/Lexer.cpp (original)
> +++ cfe/trunk/lib/Lex/Lexer.cpp Fri Mar 18 16:23:38 2011
> @@ -2091,7 +2091,7 @@
> // If the next token is obviously a // or /* */ comment, skip it efficiently
> // too (without going through the big switch stmt).
> if (CurPtr[0] == '/' && CurPtr[1] == '/' && !inKeepCommentMode() &&
> - Features.BCPLComment) {
> + Features.BCPLComment && !Features.TraditionalCPP) {
> if (SkipBCPLComment(Result, CurPtr+2))
> return; // There is a token to return.
> goto SkipIgnoredUnits;
> @@ -2280,8 +2280,10 @@
> // this as "foo / bar" and langauges with BCPL comments would lex it as
> // "foo". Check to see if the character after the second slash is a '*'.
> // If so, we will lex that as a "/" instead of the start of a comment.
> - if (Features.BCPLComment ||
> - getCharAndSize(CurPtr+SizeTmp, SizeTmp2) != '*') {
> + // However, we never do this in -traditional-cpp mode.
> + if ((Features.BCPLComment ||
> + getCharAndSize(CurPtr+SizeTmp, SizeTmp2) != '*') &&
> + !Features.TraditionalCPP) {
> if (SkipBCPLComment(Result, ConsumeChar(CurPtr, SizeTmp, Result)))
> return; // There is a token to return.
>
>
> Added: cfe/trunk/test/Preprocessor/traditional-cpp.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Preprocessor/traditional-cpp.c?rev=127910&view=auto
> ==============================================================================
> --- cfe/trunk/test/Preprocessor/traditional-cpp.c (added)
> +++ cfe/trunk/test/Preprocessor/traditional-cpp.c Fri Mar 18 16:23:38 2011
> @@ -0,0 +1,12 @@
> +/* Clang supports a very limited subset of -traditional-cpp, basically we only
> + * intend to add support for things that people actually rely on when doing
> + * things like using /usr/bin/cpp to preprocess non-source files. */
> +
> +/*
> + RUN: %clang_cc1 -traditional-cpp %s -E -o %t
> + RUN: FileCheck < %t %s
> +*/
> +
> +/* CHECK: foo // bar
> + */
> +foo // bar
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
More information about the cfe-commits
mailing list