[llvm-commits] [llvm] r110584 - in /llvm/trunk/lib/Target/ARM: ARM.td ARMConstantIslandPass.cpp ARMSubtarget.h ARMTargetMachine.cpp

[Chris Lattner]

On Aug 9, 2010, at 11:35 AM, Evan Cheng wrote:

> Author: evancheng
> Date: Mon Aug  9 13:35:19 2010
> New Revision: 110584
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=110584&view=rev
> Log:
> Change -prefer-32bit-thumb to attribute -mattr=+32bit instead to disable more 32-bit to 16-bit optimizations.

Why does this make sense to be a -mattr?  This seems like a policy choice, not a CPU attribute.  Policy decisions should be cl::opt's.

-Chris