[llvm-commits] [llvm] r83081 - /llvm/trunk/lib/CodeGen/PrologEpilogInserter.cpp

[Jim Grosbach]

On Sep 29, 2009, at 5:30 PM, Mike Stump wrote:

> On Sep 29, 2009, at 11:23 AM, Jim Grosbach wrote:
>> Author: grosbach
>> Date: Tue Sep 29 13:23:15 2009
>> New Revision: 83081
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=83081&view=rev
>> Log:
>> Simplify the tracking of virtual frame index registers. Ranges  
>> cannot overlap,
>> so a simple "current register" will suffice. Also add some additional
>> sanity-checking assertions to make sure things are as we expect.
>
> Yours:
>
> /Volumes/mrs5/net/llvm/llvm/lib/CodeGen/PrologEpilogInserter.cpp: In  
> member function ‘void llvm::PEI::scavengeFrameVirtualRegs 
> (llvm::MachineFunction&)’:
> /Volumes/mrs5/net/llvm/llvm/lib/CodeGen/PrologEpilogInserter.cpp: 
> 742: warning: ‘CurrentScratchReg’ may be used uninitialized in this  
> function
>
> ?  make ENABLE_OPTIMIZED=1 is your friend.  :-)

The warning is wrong; however, it'd take one hell of a compiler to  
figure that out. The static analyzer very likely gets it right.

r83132 should convince gcc that things are OK.