Does anybody have a spare cycle or two to review and submit this patch? Thanks!
-- Carl Confidentiality Notice: This e-mail, including all attachments, is confidential information of Lytro, Inc. If the reader of this e-mail is not the intended recipient or its authorized agent, the reader is hereby notified that any dissemination, distribution or copying of this e-mail is prohibited. If you have received this e-mail in error, please notify the sender by replying to this message and delete this e-mail immediately. On Aug 17, 2012, at 4:41 PM, Carl Norum wrote: > > Here's the change for the -G flag. I sent the mips-delay-filler one over to > llvm-dev. > > -- Carl > > > <support-g-flag.patch> > > > On Aug 17, 2012, at 4:26 PM, Carl Norum wrote: > >> >> Sure, easily done. >> >> -- Carl >> >> >> On Aug 17, 2012, at 4:18 PM, Hatanaka, Akira wrote: >> >>> Could you rename the option to "disable-mips-delay-filler" (false by >>> default)? >>> >>> ________________________________________ >>> From: Carl Norum [[email protected]] >>> Sent: Friday, August 17, 2012 4:00 PM >>> To: Hatanaka, Akira >>> Cc: Eli Friedman; cfe-commits >>> Subject: Re: [cfe-commits] Plumbing through some MIPS options >>> >>> Yeah, -G is the GCC option. We have it passed through right now in some >>> -Xassembler flags. >>> >>> I'll turn the delay filler on by default and change the other flag's name >>> to -G. >>> >>> -- Carl >>> >>> >>> On Aug 17, 2012, at 2:25 PM, Hatanaka, Akira wrote: >>> >>>> I believe this "-G" option is it: >>>> >>>> -G NUM allow referencing objects up to NUM bytes >>>> implicitly with the gp register [default 8] >>>> ________________________________________ >>>> From: Eli Friedman [[email protected]] >>>> Sent: Friday, August 17, 2012 2:20 PM >>>> To: Hatanaka, Akira >>>> Cc: Carl Norum; cfe-commits >>>> Subject: Re: [cfe-commits] Plumbing through some MIPS options >>>> >>>> On Fri, Aug 17, 2012 at 2:16 PM, Hatanaka, Akira <[email protected]> >>>> wrote: >>>>> (These are my personal opinions) >>>>> >>>>> Turning on "-enable-mips-delay-filler" by default (in >>>>> MipsDelaySlotFiller.cpp) or turning it on when compiling with "-O3" is >>>>> better. I don't think it should be exposed as a command line option. We >>>>> have been planning to change this for a while, but haven't done so. >>>>> >>>>> -mmips-short-section-threshold is fine. There is a similar option in gcc. >>>> >>>> What's the name of the gcc option? If there's precedent, we should >>>> probably follow it. >>>> >>>> -Eli >>> >> > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
