[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