Re: [patch] Remove unused variable in gnu/llvm/lib/Target/X86/X86FrameLowering.cpp

2018-06-06 Thread Todd Mortimer 
On Wed, Jun 06, 2018 at 03:57:44PM +0800, Nan Xiao wrote:
> Hi tech@,
>
> The following patch fixes the build warning:

Committed, thanks!

>
> ..
>
>  variable 'CFIIndex' [-Wunused-variable]
>   unsigned CFIIndex;
>^
> 1 warning generated.
>
>
> Index: X86FrameLowering.cpp
> ===
> RCS file: /cvs/src/gnu/llvm/lib/Target/X86/X86FrameLowering.cpp,v
> retrieving revision 1.2
> diff -u -p -r1.2 X86FrameLowering.cpp
> --- X86FrameLowering.cpp  6 Jun 2018 00:14:29 -   1.2
> +++ X86FrameLowering.cpp  6 Jun 2018 07:34:44 -
> @@ -3185,7 +3185,6 @@ void X86FrameLowering::insertReturnProte
>if (!MFI.hasReturnProtector() || !MFI.hasReturnProtectorTempRegister())
>  return;
>
> -  unsigned CFIIndex;
>MachineBasicBlock::instr_iterator MBBI = MBB.instr_begin();
>DebugLoc MBBDL = MBB.findDebugLoc(MBBI);
>const Function &Fn = MF.getFunction();
> --
> Best Regards
> Nan Xiao
>

[patch] Remove unused variable in gnu/llvm/lib/Target/X86/X86FrameLowering.cpp

2018-06-06 Thread Nan Xiao 
Hi tech@,

The following patch fixes the build warning:

..

 variable 'CFIIndex' [-Wunused-variable]
  unsigned CFIIndex;
   ^
1 warning generated.


Index: X86FrameLowering.cpp
===
RCS file: /cvs/src/gnu/llvm/lib/Target/X86/X86FrameLowering.cpp,v
retrieving revision 1.2
diff -u -p -r1.2 X86FrameLowering.cpp
--- X86FrameLowering.cpp6 Jun 2018 00:14:29 -   1.2
+++ X86FrameLowering.cpp6 Jun 2018 07:34:44 -
@@ -3185,7 +3185,6 @@ void X86FrameLowering::insertReturnProte
   if (!MFI.hasReturnProtector() || !MFI.hasReturnProtectorTempRegister())
 return;

-  unsigned CFIIndex;
   MachineBasicBlock::instr_iterator MBBI = MBB.instr_begin();
   DebugLoc MBBDL = MBB.findDebugLoc(MBBI);
   const Function &Fn = MF.getFunction();
-- 
Best Regards
Nan Xiao