[PATCH] D90421: [IndVarSimplify][LoopUtils] Move WidenIV to Utils. NFCI.
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 3 01:14:08 PST 2020
SjoerdMeijer added inline comments.
================
Comment at: llvm/include/llvm/Transforms/Utils/LoopUtils.h:466
+ // Context
+ LoopInfo *LI;
+ Loop *L;
----------------
fhahn wrote:
> Does this all need to be part of the header? Or can we hide most of that in looputils.cpp and just expose a single function to do the widening?
Thanks, I will have a go at exposing one function, and move this stuff to LoopUtils.cpp
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90421/new/
https://reviews.llvm.org/D90421
More information about the llvm-commits
mailing list