[llvm-commits] [llvm] r83122 - in /llvm/trunk: include/llvm/Target/TargetSubtarget.h lib/CodeGen/LLVMTargetMachine.cpp lib/CodeGen/PostRASchedulerList.cpp lib/Target/ARM/ARM.td lib/Target/ARM/ARMSubtarget.cpp lib/Target/ARM/ARMSubtarget.h test/CodeGen/ARM/2009-08-21-PostRAKill.ll test/CodeGen/ARM/2009-08-21-PostRAKill2.ll test/CodeGen/ARM/2009-08-21-PostRAKill3.ll test/CodeGen/ARM/2009-08-21-PostRAKill4.ll test/CodeGen/ARM/2009-09-01-PostRAProlog.ll

[Chris Lattner]

On Sep 29, 2009, at 5:10 PM, David Goodwin wrote:

> Author: david_goodwin
> Date: Tue Sep 29 19:10:16 2009
> New Revision: 83122
>
> URL: http://llvm.org/viewvc/llvm-project?rev=83122&view=rev
> Log:
> Remove -post-RA-schedule flag and add a TargetSubtarget method to  
> enable post-register-allocation scheduling. By default it is off.  
> For ARM, enable/disable with -mattr=+/-postrasched. Enable by  
> default for cortex-a8.

Hi David,

-mattr should be used for CPU attributes, not for optimization flags.   
Having a subtarget hook (or even TargetMachine hook) to indicate  
whether this should default to on or not makes sense to me, but should  
it really be controlled via -mattr?

-Chris