replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Driver: Reject -fasm-blocks except on X86 (where we just ignore it, since 

passing it is very prevalent in some circles).

llvm-svn: 116761
This commit is contained in:
Daniel Dunbar 2010-10-18 22:49:46 +00:00
parent 9bc6dbb027
commit 473f8a6f88
3 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clang/include/clang/Basic/DiagnosticDriverKinds.td
View File

@ -53,6 +53,8 @@ def err_drv_no_ast_support : Error<
"'%0': unable to use AST files with this tool">;
def err_drv_clang_unsupported : Error<
"the clang compiler does not support '%0'">;
def err_drv_clang_unsupported_per_platform : Error<
"the clang compiler does not support '%0' on this platform">;
def err_drv_clang_unsupported_opt_cxx_darwin_i386 : Error<
"the clang compiler does not support '%0' for C++ on Darwin/i386">;
def err_drv_command_failed : Error<

5
clang/include/clang/Driver/Options.td
View File

@ -237,7 +237,10 @@ def faccess_control : Flag<"-faccess-control">, Group<f_Group>;
def fallow_unsupported : Flag<"-fallow-unsupported">, Group<f_Group>;
def fapple_kext : Flag<"-fapple-kext">, Group<f_Group>;
def fasm : Flag<"-fasm">, Group<f_Group>;
def fasm_blocks : Flag<"-fasm-blocks">, Group<clang_ignored_f_Group>;
def fasm_blocks : Flag<"-fasm-blocks">, Group<f_Group>;
def fno_asm_blocks : Flag<"-fno-asm-blocks">, Group<f_Group>;
def fassume_sane_operator_new : Flag<"-fassume-sane-operator-new">, Group<f_Group>;
def fastcp : Flag<"-fastcp">, Group<f_Group>;
def fastf : Flag<"-fastf">, Group<f_Group>;

12
clang/lib/Driver/Tools.cpp
View File

@ -1469,6 +1469,18 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
options::OPT_fno_spell_checking))
CmdArgs.push_back("-fno-spell-checking");
// -fasm-blocks is disallowed except on X86, where we just ignore it.
if (Args.hasFlag(options::OPT_fasm_blocks, options::OPT_fno_asm_blocks,
false)) {
if (getToolChain().getTriple().getArch() != llvm::Triple::x86 &&
getToolChain().getTriple().getArch() != llvm::Triple::x86_64)
D.Diag(clang::diag::err_drv_clang_unsupported_per_platform)
<< "-fasm-blocks";
}
// -fasm-blocks is disallowed except on X86, where we just ignore it.
if (Arg *A = Args.getLastArg(options::OPT_fshow_overloads_EQ))
A->render(Args, CmdArgs);