[llvm-commits] [llvm] r170341 - /llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp

Tom Stellard thomas.stellard at amd.com
Mon Dec 17 07:14:53 PST 2012


Author: tstellar
Date: Mon Dec 17 09:14:53 2012
New Revision: 170341

URL: http://llvm.org/viewvc/llvm-project?rev=170341&view=rev
Log:
R600: remove nonsense setPrefLoopAlignment

The Align parameter is a power of two, so 16 results in 64K
alignment. Additional to that even 16 byte alignment doesn't
make any sense, so just remove it.

Patch by: Christian König

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
Tested-by: Michel Dänzer <michel.daenzer at amd.com>
Signed-off-by: Christian König <deathsimple at vodafone.de>

Modified:
    llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp

Modified: llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp?rev=170341&r1=170340&r2=170341&view=diff
==============================================================================
--- llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp Mon Dec 17 09:14:53 2012
@@ -217,7 +217,6 @@
 
   setSchedulingPreference(Sched::RegPressure);
   setPow2DivIsCheap(false);
-  setPrefLoopAlignment(16);
   setSelectIsExpensive(true);
   setJumpIsExpensive(true);
 





More information about the llvm-commits mailing list