[PATCH] D75381: [WinEH] Fix inttoptr+phi optimization in presence of catchswitch

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 28 14:09:46 PST 2020


hfinkel added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombinePHI.cpp:222
+  // instruction, do not do it. Similarly, do not do the transform if the value
+  // is PHI in a block with no insertion point, since we will not be able to
+  // insert a cast after the PHI.
----------------
When you say, "a block with no insertion point", I'd add, "for example, a catchswitch block." I don't think that it's otherwise obvious how one has a block with no insertion point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75381





More information about the llvm-commits mailing list