[PATCH] D113105: [InstCombine] Extend stacksave/restore elimination

Itay Bookstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 9 01:40:20 PST 2021


nextsilicon-itay-bookstein added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:1813-1814
+        for (++BI; &*BI != II; ++BI) {
+          if (Classify(&*BI) == ClassifyResult::None)
+            continue;
+
----------------
lebedev.ri wrote:
> What about stackrestore from the same stacksave?
Then it will have been eliminated when it was the subject of the enclosing switch statement, before this one was. I will add an appropriate comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113105



More information about the llvm-commits mailing list