[PATCH] D25802: [AMDGPU] Check if type transforms to i16 (VI+) when getting AMDGPUISD::FFBH_U32

Konstantin Zhuravlyov via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 13:24:14 PDT 2016


kzhuravl added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUISelLowering.cpp:2461
+  if (LegalVT != MVT::i32 &&
+        (Subtarget->getGeneration() >= AMDGPUSubtarget::VOLCANIC_ISLANDS &&
+         LegalVT != MVT::i16))
----------------
arsenm wrote:
> Subtarget->hasI16
This will involve moving `has16BitInsts()` from `SISubtarget` to `AMDGPUSubtarget`. Are there any objections?


https://reviews.llvm.org/D25802





More information about the llvm-commits mailing list