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. > Also, test? :)
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
