[PATCH] D83885: [Propeller]: Use a descriptive temporary symbol name for the end of the basic block.

Rahman Lavaee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 17 02:12:54 PDT 2020


rahmanl marked an inline comment as done.
rahmanl added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:478
 
+  MCSymbol *getEndSymbol() const;
+
----------------
tmsriram wrote:
> Can we  name it getCachedEndMCSymbol?
This naming follows the similar function MachineBasicBlock::getSymbol() which also uses a cached symbol (called CachedMCSymbol).


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

https://reviews.llvm.org/D83885





More information about the llvm-commits mailing list