[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