[llvm] [LV] Limits the splat operations be hoisted must not be defined by a recipe. (PR #117138)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 28 03:17:18 PST 2024


================
@@ -267,7 +271,11 @@ Value *VPTransformState::get(VPValue *Def, bool NeedsScalar) {
     return Data.VPV2Vector[Def];
 
   auto GetBroadcastInstrs = [this, Def](Value *V) {
-    bool SafeToHoist = Def->isDefinedOutsideLoopRegions();
+    bool SafeToHoist =
----------------
david-arm wrote:

It looks like there might be a bug in a function called `licm` in VPlanTransforms.cpp. Perhaps worth fixing that in this patch too?


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


More information about the llvm-commits mailing list