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

Ellis Hoag via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 10:30:24 PDT 2024


================
@@ -635,8 +636,12 @@ class GIMatchTableExecutor {
 
   bool shouldOptForSize(const MachineFunction *MF) const {
     const auto &F = MF->getFunction();
-    return F.hasOptSize() || F.hasMinSize() ||
----------------
ellishg wrote:

As mentioned above (but got lost in the changes), `F.hasOptSize()` 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