[cfe-commits] r144800 - in /cfe/trunk: include/clang/Driver/CC1Options.td include/clang/Driver/Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/BackendUtil.cpp lib/Driver/Tools.cpp lib/Frontend/CompilerInvocation.cpp
Chris Lattner
clattner at apple.com
Wed Nov 16 18:10:40 PST 2011
On Nov 16, 2011, at 9:34 AM, Kostya Serebryany wrote:
> Author: kcc
> Date: Wed Nov 16 11:34:26 2011
> New Revision: 144800
>
> URL: http://llvm.org/viewvc/llvm-project?rev=144800&view=rev
> Log:
> Add -f[no-]address-sanitizer flag
Great! Please document this in the user's manual,
-Chris
>
> Modified:
> cfe/trunk/include/clang/Driver/CC1Options.td
> cfe/trunk/include/clang/Driver/Options.td
> cfe/trunk/include/clang/Frontend/CodeGenOptions.h
> cfe/trunk/lib/CodeGen/BackendUtil.cpp
> cfe/trunk/lib/Driver/Tools.cpp
> cfe/trunk/lib/Frontend/CompilerInvocation.cpp
>
> Modified: cfe/trunk/include/clang/Driver/CC1Options.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CC1Options.td?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Driver/CC1Options.td (original)
> +++ cfe/trunk/include/clang/Driver/CC1Options.td Wed Nov 16 11:34:26 2011
> @@ -114,6 +114,8 @@
> HelpText<"The compilation directory to embed in the debug info.">;
> def dwarf_debug_flags : Separate<"-dwarf-debug-flags">,
> HelpText<"The string to embed in the Dwarf debug flags record.">;
> +def faddress_sanitizer: Flag<"-faddress-sanitizer">,
> + HelpText<"Enable AddressSanitizer instrumentation (memory error detection)">;
> def fforbid_guard_variables : Flag<"-fforbid-guard-variables">,
> HelpText<"Emit an error if a C++ static local initializer would need a guard variable">;
> def g : Flag<"-g">, HelpText<"Generate source level debug information">;
>
> Modified: cfe/trunk/include/clang/Driver/Options.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Driver/Options.td (original)
> +++ cfe/trunk/include/clang/Driver/Options.td Wed Nov 16 11:34:26 2011
> @@ -262,6 +262,8 @@
> def fallow_unsupported : Flag<"-fallow-unsupported">, Group<f_Group>;
> def fapple_kext : Flag<"-fapple-kext">, Group<f_Group>;
> def fapple_pragma_pack : Flag<"-fapple-pragma-pack">, Group<f_Group>;
> +def faddress_sanitizer : Flag<"-faddress-sanitizer">, Group<f_Group>;
> +def fno_address_sanitizer : Flag<"-fno-address-sanitizer">, Group<f_Group>;
> def fasm : Flag<"-fasm">, Group<f_Group>;
>
> def fasm_blocks : Flag<"-fasm-blocks">, Group<f_Group>;
>
> Modified: cfe/trunk/include/clang/Frontend/CodeGenOptions.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CodeGenOptions.h?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Frontend/CodeGenOptions.h (original)
> +++ cfe/trunk/include/clang/Frontend/CodeGenOptions.h Wed Nov 16 11:34:26 2011
> @@ -35,6 +35,7 @@
> Mixed = 2
> };
>
> + unsigned AddressSanitizer : 1; /// Enable AddressSanitizer.
> unsigned AsmVerbose : 1; /// -dA, -fverbose-asm.
> unsigned ObjCAutoRefCountExceptions : 1; /// Whether ARC should be EH-safe.
> unsigned CUDAIsDevice : 1; /// Set when compiling for CUDA device.
> @@ -151,6 +152,7 @@
>
> public:
> CodeGenOptions() {
> + AddressSanitizer = 0;
> AsmVerbose = 0;
> CUDAIsDevice = 0;
> CXAAtExit = 1;
>
> Modified: cfe/trunk/lib/CodeGen/BackendUtil.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/BackendUtil.cpp?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/lib/CodeGen/BackendUtil.cpp (original)
> +++ cfe/trunk/lib/CodeGen/BackendUtil.cpp Wed Nov 16 11:34:26 2011
> @@ -115,6 +115,11 @@
> PM.add(createObjCARCOptPass());
> }
>
> +static void addAddressSanitizerPass(const PassManagerBuilder &Builder,
> + PassManagerBase &PM) {
> + PM.add(createAddressSanitizerPass());
> +}
> +
> void EmitAssemblyHelper::CreatePasses() {
> unsigned OptLevel = CodeGenOpts.OptimizationLevel;
> CodeGenOptions::InliningMethod Inlining = CodeGenOpts.Inlining;
> @@ -141,6 +146,11 @@
> PMBuilder.addExtension(PassManagerBuilder::EP_ScalarOptimizerLate,
> addObjCARCOptPass);
> }
> +
> + if (CodeGenOpts.AddressSanitizer) {
> + PMBuilder.addExtension(PassManagerBuilder::EP_ScalarOptimizerLate,
> + addAddressSanitizerPass);
> + }
>
> // Figure out TargetLibraryInfo.
> Triple TargetTriple(TheModule->getTargetTriple());
>
> Modified: cfe/trunk/lib/Driver/Tools.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Driver/Tools.cpp (original)
> +++ cfe/trunk/lib/Driver/Tools.cpp Wed Nov 16 11:34:26 2011
> @@ -1701,6 +1701,10 @@
> if (getToolChain().SupportsProfiling())
> Args.AddLastArg(CmdArgs, options::OPT_pg);
>
> + if (Args.hasFlag(options::OPT_faddress_sanitizer,
> + options::OPT_fno_address_sanitizer, false))
> + CmdArgs.push_back("-faddress-sanitizer");
> +
> // -flax-vector-conversions is default.
> if (!Args.hasFlag(options::OPT_flax_vector_conversions,
> options::OPT_fno_lax_vector_conversions))
>
> Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=144800&r1=144799&r2=144800&view=diff
> ==============================================================================
> --- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original)
> +++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Wed Nov 16 11:34:26 2011
> @@ -139,6 +139,8 @@
> Res.push_back("-dwarf-debug-flags");
> Res.push_back(Opts.DwarfDebugFlags);
> }
> + if (Opts.AddressSanitizer)
> + Res.push_back("-faddress-sanitizer");
> if (Opts.ObjCRuntimeHasARC)
> Res.push_back("-fobjc-runtime-has-arc");
> if (Opts.ObjCRuntimeHasTerminate)
> @@ -1049,6 +1051,7 @@
> Opts.UnrollLoops = Args.hasArg(OPT_funroll_loops) ||
> (Opts.OptimizationLevel > 1 && !Opts.OptimizeSize);
>
> + Opts.AddressSanitizer = Args.hasArg(OPT_faddress_sanitizer);
> Opts.AsmVerbose = Args.hasArg(OPT_masm_verbose);
> Opts.ObjCAutoRefCountExceptions = Args.hasArg(OPT_fobjc_arc_exceptions);
> Opts.ObjCRuntimeHasARC = Args.hasArg(OPT_fobjc_runtime_has_arc);
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
More information about the cfe-commits
mailing list