[PATCH] D84601: [InstCombine] Fold freeze into phi if beneficial

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 26 17:49:51 PDT 2020


aqjune marked 3 inline comments as done.
aqjune added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/freeze-phi.ll:171
   ret i32 %y.fr
 }
----------------
nikic wrote:
> Can you also add a test with an all-constant phi, in which case the freeze will get dropped completely? I believe the code also handles that case.
Seems like InstSimplify already deals with the case: function `const` in this file has a phi with constants.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84601





More information about the llvm-commits mailing list