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
