(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. ________________________________________ From: Eli Friedman [[email protected]] Sent: Friday, August 17, 2012 1:55 PM To: Carl Norum; Hatanaka, Akira Cc: cfe-commits Subject: Re: [cfe-commits] Plumbing through some MIPS options On Fri, Aug 17, 2012 at 1:08 PM, Carl Norum <[email protected]> wrote: > > Over in this thread on the llvm-dev list: > > http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-August/052574.html > > It was suggested that I file bugs about having to pass some MIPS-specific > flags through via "-Xclang" and "-mllvm". I got a pointer to the right area > of the tree, though, and came up with the attached patch. > > Can someone let me know if it looks good and submit it if possible? Maybe -mmips-short-section-threshold? Saving characters isn't really a priority here. Someone working on the MIPS backend should comment on whether it makes sense to expose these options as-is, especially -enable-mips-delay-filler. -Eli _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
