[PATCH] D84944: [JumpThreading] Let SimplifyPartiallyRedundantLoad look into freeze

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 30 12:57:36 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/JumpThreading.cpp:1181
+    // Look into freeze's operand
+    SimplifyValue = FI->getOperand(0);
+
----------------
Am I understanding correctly, that SimplifyPartiallyRedundantLoad is actually a legal transform for arbitrary load instructions, but we have a heuristic so we only perform it for "important" instructions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84944



More information about the llvm-commits mailing list