[PATCH] Adding option -fno-inline-asm to disallow inline asm

Steven Wu stevenwu at apple.com
Fri Jan 16 13:52:28 PST 2015


- Remove -fno-inline-asm option and do not disallow asm label

  I agree that it is not a good idea disallow all asm labels with the new option. It is probably a Sema check should be implemented later to distinguish different asm labels and the type of occurrence (like in a system header).

  I don't like the part of code using -fno-inline-asm to imply the other two very much as well. I have no problem removing that flag.


http://reviews.llvm.org/D6870

Files:
  include/clang/Basic/DiagnosticParseKinds.td
  include/clang/Basic/LangOptions.def
  include/clang/Driver/Options.td
  lib/Driver/Tools.cpp
  lib/Frontend/CompilerInvocation.cpp
  lib/Parse/ParseStmtAsm.cpp
  test/Driver/inline-asm.c
  test/Driver/ms-inline-asm.c
  test/Parser/ms-inline-asm.c
  test/Parser/no-gnu-inline-asm.c

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6870.18318.patch
Type: text/x-patch
Size: 5287 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150116/f5a87423/attachment.bin>


More information about the cfe-commits mailing list