[PATCH] D141134: [NFC] Only expose getXXXSize functions in TypeSize

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/include/llvm/Support/TypeSize.h:319-322
+  // Make 'getKnownMinValue' / 'getFixedValue' private as they are exposed as
+  // 'getKnownMinValue' / 'getFixedValue' lower in this class.
+  constexpr ScalarTy getKnownMinValue() const { return UP::getKnownMinValue(); }
+  constexpr ScalarTy getFixedValue() const { return UP::getFixedValue(); }
----------------
They're both only used once below, so I think you can remove the private interface and just call the parent class' method directly?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141134