[llvm-commits] [llvm] r118092 - in /llvm/trunk/lib/Target/MBlaze: MBlazeInstrFPU.td MBlazeInstrInfo.td

Frits van Bommel fvbommel at gmail.com
Tue Nov 2 16:37:59 PDT 2010


On Tue, Nov 2, 2010 at 11:26 PM, Chris Lattner <sabre at nondot.org> wrote:
> Author: lattner
> Date: Tue Nov  2 17:26:33 2010
> New Revision: 118092
>
> URL: http://llvm.org/viewvc/llvm-project?rev=118092&view=rev
> Log:
> mark a few codegenonly instructions.

Since these all use MBlazePseudo, and all MBlazePseudo-derived
instructions now set isCodegenOnly = 1, wouldn't it make more sense to
set it in once in that class? Or automatically set it in it's
superclass, like you did for X86 and ARM in r117862[1]?



[1]: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20101025/110856.html




More information about the llvm-commits mailing list