[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