[llvm-commits] [llvm] r114064 - in /llvm/trunk: include/llvm/MC/MCInst.h lib/Target/ARM/ARMMCInstLower.cpp lib/Target/ARM/AsmPrinter/ARMInstPrinter.cpp lib/Target/ARM/Disassembler/ARMDisassemblerCore.cpp

[Jim Grosbach]

On Sep 16, 2010, at 10:46 AM, Jim Grosbach wrote:

> 
> On Sep 16, 2010, at 10:44 AM, Chris Lattner wrote:
> 
>> 
>> On Sep 15, 2010, at 8:45 PM, Jim Grosbach wrote:
>> 
>>> +  const double &getFPImm() const {
>>>   assert(isFPImm() && "This is not an FP immediate");
>>>   return FPImmVal;
>>> }
>> 
>> This can just return it by value.
> 
> 
> Woops! Quite right, of course. Thanks for catching that. Fixed in r114099.

And fixed a follow-up brain fart in r114100.... *blush*