[cfe-commits] Plumbing through some MIPS options

[Hatanaka, Akira]

(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 [eli.friedman at gmail.com]
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 <carl at lytro.com> 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