[llvm] [AMDGPU][LRO] LRO fix PHI same-BB filter; treat i8/i16 binops as profitable (PR #155800)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 28 03:02:52 PDT 2025


================
@@ -126,7 +126,37 @@ class LiveRegOptimizer {
     return LK.first != TargetLoweringBase::TypeLegal;
   }
 
-  bool isOpLegal(Instruction *I) { return isa<StoreInst, IntrinsicInst>(I); }
+  bool isOpLegal(Instruction *I) {
+    if (auto *Intr = dyn_cast<IntrinsicInst>(I))
+      return true; // FIXME: narrow to known native intrinsics (DOT/MFMA/tbuffer) or use TTI cost.
----------------
arsenm wrote:

Can't TTI directly handle the entire function already? 

https://github.com/llvm/llvm-project/pull/155800


More information about the llvm-commits mailing list