On Dec 5, 2012, at 3:20 PM, Richard Smith <[email protected]> wrote:
> On Wed, Dec 5, 2012 at 3:20 PM, Richard Smith <[email protected]> wrote: > On Wed, Dec 5, 2012 at 3:08 PM, Chad Rosier <[email protected]> wrote: > Author: mcrosier > Date: Wed Dec 5 17:08:09 2012 > New Revision: 169445 > > URL: http://llvm.org/viewvc/llvm-project?rev=169445&view=rev > Log: > [driver, ms-inline asm] Have -fms-extensions enable the AsmBlocks language > option. MS-style inline asm can now be enabled by either -fasm-blocks or > -fms-extensions. > rdar://12808010 > > Modified: > cfe/trunk/lib/Frontend/CompilerInvocation.cpp > cfe/trunk/test/Parser/MicrosoftExtensions.c > > Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=169445&r1=169444&r2=169445&view=diff > ============================================================================== > --- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original) > +++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Wed Dec 5 17:08:09 2012 > @@ -1172,7 +1172,7 @@ > Opts.MicrosoftExt > = Args.hasArg(OPT_fms_extensions) || Args.hasArg(OPT_fms_compatibility); > Opts.MicrosoftMode = Args.hasArg(OPT_fms_compatibility); > - Opts.AsmBlocks = Args.hasArg(OPT_fasm_blocks); > + Opts.AsmBlocks = Args.hasArg(OPT_fasm_blocks) || > Args.hasArg(OPT_fms_extensions); > > Presumably this should be || Opts.MicrosoftExt? -fms-compatibility should > enable this feature too. See r169445. Thanks, Richard. > > Also, test? :) There was a test case included in the original commit. Chad
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
