[PATCH] D19160: AMDGPU: Fold shift into cvt_f32_ubyteN

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 07:38:02 PDT 2016


tstellarAMD added inline comments.

================
Comment at: lib/Target/AMDGPU/SIISelLowering.cpp:2678-2681
@@ +2677,6 @@
+
+    if (Src.getOpcode() == ISD::SRL) {
+      // cvt_f32_ubyte0 (shl x, 16) -> cvt_f32_ubyte2 x
+      // cvt_f32_ubyte1 (shl x, 16) -> cvt_f32_ubyte3 x
+      // cvt_f32_ubyte0 (shl x, 8) -> cvt_f32_ubyte1 x
+
----------------
The comment doesn't match the if statement.  Which is correct?


http://reviews.llvm.org/D19160





More information about the llvm-commits mailing list