[PATCH] D74890: [Analysis] getParentLoop() documentation
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 20 03:57:15 PST 2020
fhahn added inline comments.
================
Comment at: llvm/include/llvm/Analysis/LoopInfo.h:106
BlockT *getHeader() const { return getBlocks().front(); }
+ // Return the parent loop of this loop (i.e. the loop in which
+ // this loop is entirely contained). If there's no such loop (i.e.
----------------
fhahn wrote:
> Use `///`. That's how doxygen comments are defined. It might also be worth clarifying the parent loop relationship in https://llvm.org/docs/LoopTerminology.html.
>
> For the comment here I would suggest to start with a brief first sentence and maybe another one that explain the parent relationship, like
>
> Return the parent loop if it exists or nullptr otherwise.
> Use ///. That's how doxygen comments are defined. It might also be worth clarifying the parent loop relationship in https://llvm.org/docs/LoopTerminology.html.
Reading this back I realized it might sound a bit harsh. That was not the intention, sorry!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74890/new/
https://reviews.llvm.org/D74890
More information about the llvm-commits
mailing list