[PATCH] D139905: [NFC] Cleanup: BasicBlock::getInstList() is now private.

[Vasileios Porpodas via Phabricator via llvm-commits]

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


================
Comment at: llvm/include/llvm/IR/BasicBlock.h:385
   /// Returns a pointer to a member of the instruction list.
-  static InstListType BasicBlock::*getSublistAccess(Instruction*) {
+  static InstListType BasicBlock::*getSublistAccess(Instruction *) {
     return &BasicBlock::InstList;
----------------
aeubanks wrote:
> this should also be private, although I'd hope people don't abuse this
> 
> looks like it's used as part of `ilist_node_with_parent`, maybe friend class that?
Yeah good point. I made this private too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139905