search for: constructjob

On Tue, Oct 1, 2013 at 3:53 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote: > The option handling in clang in fairly different from opt. The comment > about the mixed name was just a guess as to why you don't see the > driver passing it down to -cc1. Clang::ConstructJob is ~2k lines long. I was putting the handling of this option too far down. I moved it up and it's now being passed. Not sure what order needs to be kept in this function, however. > Once that is working, you will probably need to: > > * Patch ParseCodeGenArgs to record the option &g...

...cortex-m3 I would like to understand what -ccc-gcc-name does exactly and how can I get ride of it? >From what I understand, it's needed by the driver in order to find the binutils for calling the assembler and the linker. Is that correct? After some grep'ing I ended up in gcc::Common::ConstructJob, where the option value is retrieved, but I'm not sure how does it fit with the rest. How else can I tell llvm where to look for binutils? Hope someone can save me a few hours around the codebase with some insights. Regards, Salvatore

...0xf689b704 sigacthandler + 88 5 libc.so.1 0xf68acf50 __lwp_sigqueue + 8 6 libc.so.1 0xf682988c abort + 200 7 libLLVM-3.7.so 0xfef61318 llvm::llvm_unreachable_internal(char const*, char const*, unsigned int) + 376 8 clang 0x00888a94 clang::driver::tools::solaris::Linker::ConstructJob(clang::driver::Compilati on&, clang::driver::JobAction const&, clang::driver::InputInfo const&, llvm::SmallVector<clang::driver::InputInfo, 4u> const&, llvm::opt::ArgList const&, char const*) const + 1348 9 clang 0x00835938 clang::driver::Driver::BuildJobsForAct...

...ature">; def fbootclasspath_EQ : Joined<["-"], "fbootclasspath=">, Group<f_Group>; diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp index dd48bc1..729da37 100644 --- a/lib/Driver/Tools.cpp +++ b/lib/Driver/Tools.cpp @@ -3594,6 +3594,11 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, D.Diag(diag::err_drv_argument_not_allowed_with) << "-fomit-frame-pointer" << A->getAsString(Args); + if (Args.hasArg(options::OPT_fauto_profile)) { + CmdArgs.push_back("-auto-profile"); + } + //...

The option handling in clang in fairly different from opt. The comment about the mixed name was just a guess as to why you don't see the driver passing it down to -cc1. Once that is working, you will probably need to: * Patch ParseCodeGenArgs to record the option * Patch EmitAssemblyHelper::CreatePasses to set the option to enable the pass in the pass manager (assuming that is the effect

...013 at 3:53 PM, Rafael Espíndola > <rafael.espindola at gmail.com> wrote: >> The option handling in clang in fairly different from opt. The comment >> about the mixed name was just a guess as to why you don't see the >> driver passing it down to -cc1. > > Clang::ConstructJob is ~2k lines long. I was putting the handling of > this option too far down. I moved it up and it's now being passed. > Not sure what order needs to be kept in this function, however. > >> Once that is working, you will probably need to: >> >> * Patch ParseCodeGenAr...

...hat -ccc-gcc-name does exactly and how can I get > ride of it? > > >From what I understand, it's needed by the driver in order to find the > >binutils > for calling the assembler and the linker. Is that correct? > > After some grep'ing I ended up in gcc::Common::ConstructJob, where the > option value is retrieved, but I'm not sure how does it fit with the rest. > > How else can I tell llvm where to look for binutils? > > Hope someone can save me a few hours around the codebase with some > insights. > > Regards, > Salvatore > _______...

...otclasspath_EQ : Joined<["-"], "fbootclasspath=">, Group<f_Group>; > diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp > index dd48bc1..729da37 100644 > --- a/lib/Driver/Tools.cpp > +++ b/lib/Driver/Tools.cpp > @@ -3594,6 +3594,11 @@ void Clang::ConstructJob(Compilation &C, const > JobAction &JA, > D.Diag(diag::err_drv_argument_not_allowed_with) > << "-fomit-frame-pointer" << A->getAsString(Args); > > + if (Args.hasArg(options::OPT_fauto_profile)) { > + CmdArgs.push_back("-auto...

On Tue, Oct 1, 2013 at 3:42 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote: > You are calling the option -auto-profile in some places and > -fauto-profile in others. Maybe it is just a typo? In opt, the option is named '-auto-profile' when I instantiate the pass: INITIALIZE_PASS(AutoProfile, "auto-profile", "Auto Profile loader", false,

...Args.push_back("__llvm__safestack_init"); +} + static SmallString<128> getSanitizerRTLibName(const ToolChain &TC, StringRef Sanitizer, bool Shared) { @@ -3675,7 +3698,14 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA, // -stack-protector=0 is default. unsigned StackProtectorLevel = 0; - if (Arg *A = Args.getLastArg(options::OPT_fno_stack_protector, + if (Args.hasFlag(options::OPT_fsafe_stack, + options::OPT_fno_safe_stack, false)) { + St...

On Thu, Jun 4, 2015 at 5:33 PM, Reid Kleckner <rnk at google.com> wrote: > On Thu, Jun 4, 2015 at 5:17 PM, Teresa Johnson <tejohnson at google.com> wrote: >> >> Agreed. Although I assume you mean invoke the new pass under a >> ThinLTO-only option so that avail extern are not dropped in the >> compile pass before the LTO link? > > > No, this pass

On Thu, Jun 4, 2015 at 3:58 PM, Duncan P. N. Exon Smith < dexonsmith at apple.com> wrote: > > > Personally, I think the right approach is to add a bool to > createGlobalDCEPass defaulting to true named something like > IsAfterInlining. In most standard pass pipelines, GlobalDCE runs after > inlining for obvious reasons, so the default makes sense. The special case > is

...Opts.RerollLoops; PMBuilder.addExtension(PassManagerBuilder::EP_EarlyAsPossible, Index: lib/Driver/Tools.cpp =================================================================== --- lib/Driver/Tools.cpp (revision 237590) +++ lib/Driver/Tools.cpp (working copy) @@ -2676,6 +2676,10 @@ void Clang::ConstructJob(Compilation &C, const Job assert((isa<CompileJobAction>(JA) || isa<BackendJobAction>(JA)) && "Invalid action for clang tool."); + if (JA.getType() == types::TY_LTO_IR || + JA.getType() == types::TY_LTO_BC) { + CmdArgs.push_back(&qu...