[llvm] r206501 - R600: Add comment clariying use of sext for result of MUL_U24
Tom Stellard
thomas.stellard at amd.com
Thu Apr 17 14:00:14 PDT 2014
Author: tstellar
Date: Thu Apr 17 16:00:13 2014
New Revision: 206501
URL: http://llvm.org/viewvc/llvm-project?rev=206501&view=rev
Log:
R600: Add comment clariying use of sext for result of MUL_U24
Modified:
llvm/trunk/lib/Target/R600/AMDGPUISelLowering.cpp
Modified: llvm/trunk/lib/Target/R600/AMDGPUISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/R600/AMDGPUISelLowering.cpp?rev=206501&r1=206500&r2=206501&view=diff
==============================================================================
--- llvm/trunk/lib/Target/R600/AMDGPUISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/R600/AMDGPUISelLowering.cpp Thu Apr 17 16:00:13 2014
@@ -1158,6 +1158,8 @@ SDValue AMDGPUTargetLowering::PerformDAG
break;
}
+ // We need to use sext even for MUL_U24, because MUL_U24 is used
+ // for signed multiply of 8 and 16-bit types.
SDValue Reg = DAG.getSExtOrTrunc(Mul, DL, VT);
return Reg;
More information about the llvm-commits
mailing list