[PATCH] D99490: [NFC][LoopUnswitch] Move hasPartialIVCondition to LoopUtils

JinGu Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 29 03:29:37 PDT 2021


jaykang10 added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Utils/LoopUtils.h:516
+/// the condition in the loops created for the true or false successors.
+Optional<IVConditionInfo> hasPartialIVCondition(Loop *L, unsigned MSSAThreshold,
+                                                MemorySSA *MSSA, AAResults *AA);
----------------
fhahn wrote:
> `L`, `MSSA` and `AA` are all needed. Can we pass them by reference?
Ah, let me change it on https://reviews.llvm.org/D99354. Is it ok?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99490



More information about the llvm-commits mailing list