What build problem did this cause? Did this just not compile (it looks as if ArgList has hasArg and hasFlag, so I'm not sure what the specific problem might've been)
On Wed, Jul 13, 2016 at 11:45 PM Dean Michael Berris via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: dberris > Date: Thu Jul 14 01:37:46 2016 > New Revision: 275377 > > URL: http://llvm.org/viewvc/llvm-project?rev=275377&view=rev > Log: > Use hasFlag instead of hasArg > > Summary: Fix the build to use hasFlag instead of hasArg for checking some > flags. > > Reviewers: echristo > > Subscribers: mehdi_amini, cfe-commits > > Differential Revision: http://reviews.llvm.org/D22338 > > Modified: > cfe/trunk/lib/Driver/Tools.cpp > > Modified: cfe/trunk/lib/Driver/Tools.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=275377&r1=275376&r2=275377&view=diff > > ============================================================================== > --- cfe/trunk/lib/Driver/Tools.cpp (original) > +++ cfe/trunk/lib/Driver/Tools.cpp Thu Jul 14 01:37:46 2016 > @@ -4609,8 +4609,8 @@ void Clang::ConstructJob(Compilation &C, > > Args.AddAllArgs(CmdArgs, options::OPT_finstrument_functions); > > - if (Args.hasArg(options::OPT_fxray_instrument, > - options::OPT_fnoxray_instrument, false)) { > + if (Args.hasFlag(options::OPT_fxray_instrument, > + options::OPT_fnoxray_instrument, false)) { > CmdArgs.push_back("-fxray-instrument"); > if (Arg *A = > Args.getLastArg(options::OPT_fxray_instruction_threshold_, > > options::OPT_fxray_instruction_threshold_EQ)) { > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits