[llvm] ea438b4 - [X86] Inline variable to avoid unused warning in Release builds. NFCI.

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Fri May 21 09:33:25 PDT 2021



> On May 21, 2021, at 17:29, Benjamin Kramer via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> 
> Author: Benjamin Kramer
> Date: 2021-05-21T18:28:46+02:00
> New Revision: ea438b489828f717db9bb5433863854c1f61c7d7
> 
> URL: https://github.com/llvm/llvm-project/commit/ea438b489828f717db9bb5433863854c1f61c7d7
> DIFF: https://github.com/llvm/llvm-project/commit/ea438b489828f717db9bb5433863854c1f61c7d7.diff
> 
> LOG: [X86] Inline variable to avoid unused warning in Release builds. NFCI.
> 
> Added: 
> 
> 
> Modified: 
>    llvm/lib/Target/X86/X86ExpandPseudo.cpp
> 
> Removed: 
> 
> 
> 
> ################################################################################
> diff  --git a/llvm/lib/Target/X86/X86ExpandPseudo.cpp b/llvm/lib/Target/X86/X86ExpandPseudo.cpp
> index baeebf94a683..88e34911a7df 100644
> --- a/llvm/lib/Target/X86/X86ExpandPseudo.cpp
> +++ b/llvm/lib/Target/X86/X86ExpandPseudo.cpp
> @@ -196,8 +196,7 @@ void X86ExpandPseudo::expandCALL_RVMARKER(MachineBasicBlock &MBB,
>   MachineInstr &MI = *MBBI;
> 
>   MachineInstr *OriginalCall;
> -  MachineOperand &CallTarget = MI.getOperand(1);
> -  assert((CallTarget.isGlobal() || CallTarget.isReg()) &&
> +  assert((MI.getOperand(1).isGlobal() || MI.getOperand(1).isReg()) &&
>          "invalid operand for regular call");
>   unsigned Opc = -1;
>   if (MI.getOpcode() == X86::CALL64m_RVMARKER)
> 



Thanks!


More information about the llvm-commits mailing list