[llvm-commits] [llvm] r46394 - in /llvm/trunk/lib: Target/PowerPC/PPCAsmPrinter.cpp Transforms/Scalar/SimplifyCFG.cpp

Chris Lattner clattner at apple.com
Sun Jan 27 20:37:16 PST 2008


>>> +      BuildMI(MBB, MBB.end(), TM.getInstrInfo()->get(PPC::NOP));
>>> +  }
>>
>> Is there any reason to actually make a machineinstr here?  It seems
>> like it would be possible to just emit "nop" to the .s file.  The JIT
>> doesn't need this.
>>
> I initially had that, but I thought that it would be better to do
> things though the "print machine instruction" function just so I got
> the syntax correct. But nop is probably safe to emit like that.

"nop" is valid on arm, thanks.

-Chris




More information about the llvm-commits mailing list