[llvm] e2c27fd - [X86] X86FixupInstTuning - hoist OptSize flag. NFC.

[Simon Pilgrim via llvm-commits]

Author: Simon Pilgrim
Date: 2025-06-13T11:11:01+01:00
New Revision: e2c27fd66a13c7a37cccbf4309532fcbce86c09b

URL: https://github.com/llvm/llvm-project/commit/e2c27fd66a13c7a37cccbf4309532fcbce86c09b
DIFF: https://github.com/llvm/llvm-project/commit/e2c27fd66a13c7a37cccbf4309532fcbce86c09b.diff

LOG: [X86] X86FixupInstTuning - hoist OptSize flag. NFC.

Allow reuse in a future patch.

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86FixupInstTuning.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86FixupInstTuning.cpp b/llvm/lib/Target/X86/X86FixupInstTuning.cpp
index ce1e4966553f5..8c1ff523c975a 100644
--- a/llvm/lib/Target/X86/X86FixupInstTuning.cpp
+++ b/llvm/lib/Target/X86/X86FixupInstTuning.cpp
@@ -81,6 +81,7 @@ bool X86FixupInstTuningPass::processInstruction(
   MachineInstr &MI = *I;
   unsigned Opc = MI.getOpcode();
   unsigned NumOperands = MI.getDesc().getNumOperands();
+  bool OptSize = MF.getFunction().hasOptSize();
 
   auto GetInstTput = [&](unsigned Opcode) -> std::optional<double> {
     // We already checked that SchedModel exists in `NewOpcPreferable`.
@@ -226,8 +227,7 @@ bool X86FixupInstTuningPass::processInstruction(
                                unsigned MovImm) -> bool {
     if ((MI.getOperand(NumOperands - 1).getImm() & Mask) != MovImm)
       return false;
-    bool Force = MF.getFunction().hasOptSize();
-    if (!Force && !NewOpcPreferable(MovOpc))
+    if (!OptSize && !NewOpcPreferable(MovOpc))
       return false;
     MI.setDesc(TII->get(MovOpc));
     MI.removeOperand(NumOperands - 1);