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
