[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

Kostya Serebryany kcc at google.com
Wed Nov 16 18:12:09 PST 2011


On Wed, Nov 16, 2011 at 6:10 PM, Chris Lattner <clattner at apple.com> wrote:

>
> 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,
>

Probably, not before it actually becomes usable.
(We still need the run-time library and then the linker flags; working on
that)
--kcc


>
> -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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111116/2ba84bf5/attachment.html>


More information about the cfe-commits mailing list