[llvm-commits] [llvm] r142530 - in /llvm/trunk: lib/Target/ARM/ARMISelLowering.cpp test/CodeGen/ARM/2011-10-18-DisableMovtSize.ll

[Chris Lattner]

On Oct 26, 2011, at 9:50 AM, Owen Anderson wrote:

> 
> On Oct 26, 2011, at 12:37 AM, James Molloy wrote:
> 
>> I also heavily dislike the Darwin-specific stuff in the backend and the
>> whole idea of "We don't want to do this for Darwin" - if something is a
>> negative tradeoff, have an appropriate target-independent knob that can be
>> turned to a different position for Darwin.
> 
> +1
> 
> I think the real issue here is that we don't have a way to distinguish -Os vs. -Oz at the IR level.  Perhaps we need a new function attribute to distinguish the two, though I don't know a good name for it offhand.  Then "We don't want this for Darwin" just becomes a policy decision in the driver.

I think we just want another attribute, just like OptimizeForSize but the Oz variant.  Maybe OptimizeForSizeHarder ;-)

-Chris