[llvm] [SandboxVec][BottomUpVec] Add cost estimation and tr-accept-or-revert pass (PR #124620)

Jorge Gorbe Moya via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 7 14:34:38 PST 2025


================
@@ -387,79 +374,29 @@ Value *BottomUpVec::vectorizeRec(ArrayRef<Value *> Bndl,
 }
 
 bool BottomUpVec::tryVectorize(ArrayRef<Value *> Bndl) {
+  Change = false;
   DeadInstrCandidates.clear();
   Legality->clear();
   vectorizeRec(Bndl, {}, /*Depth=*/0);
   tryEraseDeadInstrs();
   return Change;
 }
 
-bool BottomUpVec::runOnFunction(Function &F, const Analyses &A) {
+bool BottomUpVec::runOnRegion(Region &Rgn, const Analyses &A) {
+  const auto &SeedSlice = Rgn.getAux();
----------------
slackito wrote:

Once we've retrieved the seed slice from the region, does it make sense to keep it there? Should there be a `Rgn.clearAux` method?

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


More information about the llvm-commits mailing list