replace extract-element at its original location

Gang Y Chen · igcbot · commit ac0ac23f8257 · 2023-07-28T19:17:32.000+02:00
to preserve load-clustering effect
diff --git a/IGC/Compiler/CISACodeGen/VectorPreProcess.cpp b/IGC/Compiler/CISACodeGen/VectorPreProcess.cpp
@@ -1550,10 +1550,11 @@ Instruction* VectorPreProcess::simplifyLoadStore(Instruction* Inst)
                 unsigned Idx = int_cast<unsigned>(CI->getZExtValue());
                 if (NewEEI[Idx] == nullptr)
                 {
-                    NewEEI[Idx] = Builder.CreateExtractElement(NewLI, CI);
+                    // To preserve the order, insert next to the original EEI
+                    NewEEI[Idx] = ExtractElementInst::Create(NewLI, CI, "", EEI);
                     if (BC)
                     {
-                        NewBC[Idx] = Builder.CreateBitCast(NewEEI[Idx], DstEltTy);
+                        NewBC[Idx] = CastInst::Create(BC->getOpcode(), NewEEI[Idx], DstEltTy, "", EEI);
                         dyn_cast<BitCastInst>(NewBC[Idx])->setDebugLoc(BC->getDebugLoc());
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -1550,10 +1550,11 @@ Instruction* VectorPreProcess::simplifyLoadStore(Instruction* Inst)`
`1550`	`1550`	`unsigned Idx = int_cast<unsigned>(CI->getZExtValue());`
`1551`	`1551`	`if (NewEEI[Idx] == nullptr)`
`1552`	`1552`	`{`
`1553`		`- NewEEI[Idx] = Builder.CreateExtractElement(NewLI, CI);`
	`1553`	`+ // To preserve the order, insert next to the original EEI`
	`1554`	`+ NewEEI[Idx] = ExtractElementInst::Create(NewLI, CI, "", EEI);`
`1554`	`1555`	`if (BC)`
`1555`	`1556`	`{`
`1556`		`- NewBC[Idx] = Builder.CreateBitCast(NewEEI[Idx], DstEltTy);`
	`1557`	`+ NewBC[Idx] = CastInst::Create(BC->getOpcode(), NewEEI[Idx], DstEltTy, "", EEI);`
`1557`	`1558`	`dyn_cast<BitCastInst>(NewBC[Idx])->setDebugLoc(BC->getDebugLoc());`
`1558`	`1559`	`}`
`1559`	`1560`	`}`