[PATCH] D116696: [SCEVExpand] Do not create redundant 'or false' for pred expansion.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 6 07:45:20 PST 2022


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp:2590
+    else
+      Check = Builder.CreateOr(Check, NextCheck);
   }
----------------
reames wrote:
> If you wanted, there's also a CreateOr version which takes an arrayref of operands and does this expansion for you.  
Thanks, the committed version uses this version.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116696



More information about the llvm-commits mailing list