[PATCH] D107219: [LICM] Support sinking in LNICM
Bardia Mahjour via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 9 09:17:28 PDT 2021
bmahjour added inline comments.
================
Comment at: llvm/include/llvm/Transforms/Utils/LoopUtils.h:155
+ SinkAndHoistLICMFlags &, OptimizationRemarkEmitter *,
+ Loop * = nullptr);
+
----------------
Please name the parameters of this function and add a comment about what each parameter means. Someone reading this declaration and its comments will see two `Loop *` being passed and cannot figure out what they are without going through its implementation.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107219/new/
https://reviews.llvm.org/D107219
More information about the llvm-commits
mailing list