[PATCH] D154205: [MachineLICM] Handle subloops

Benjamin Kramer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 04:27:20 PDT 2023


bkramer added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:801
+        // pressure because we have already visited inner loop's preheader.
         UpdateRegPressure(&MI);
+      }
----------------
When MI is hoisted the pointer is no longer valid. I'm seeing use after frees with asan after this change, so reverted in 3454cf6


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154205/new/

https://reviews.llvm.org/D154205



More information about the llvm-commits mailing list