[llvm] Remove llvm::shouldOptForSize() from Utils.h (PR #112630)

Ellis Hoag via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 22:50:25 PDT 2024


================
@@ -1618,13 +1618,6 @@ int64_t llvm::getICmpTrueVal(const TargetLowering &TLI, bool IsVector,
   llvm_unreachable("Invalid boolean contents");
 }
 
-bool llvm::shouldOptForSize(const MachineBasicBlock &MBB,
-                            ProfileSummaryInfo *PSI, BlockFrequencyInfo *BFI) {
-  const auto &F = MBB.getParent()->getFunction();
-  return F.hasOptSize() || F.hasMinSize() ||
----------------
ellishg wrote:

`F.hasOptSize()` (now called in `shouldOptimizeForSize()`) already calls `hasMinSize()`.
https://github.com/llvm/llvm-project/blob/927af63fddb8e34f23b2974f812156767988ec5f/llvm/include/llvm/IR/Function.h#L707-L709

https://github.com/llvm/llvm-project/pull/112630


More information about the llvm-commits mailing list