[llvm-commits] [llvm] r127097 - /llvm/trunk/include/llvm/CodeGen/MachineInstr.h

Anton Korobeynikov asl at math.spbu.ru
Tue Mar 8 13:29:31 PST 2011

Hello Micah,

>  Is there a reason why this is being shortened to 8 bits?
Yes. See next commits. Basically the field was splited in order to
introduce the MIFlags field and not to increase the size of
MachineInstr class.
Given that currently the only user of AsmPrinter flags occupied only 1
bit this was considered as a safe procedure.

> I currently need all 16 bits in AMD's backend and this would break our backend if we sync to past this changelist.
Well, you can always increase the size of MachineInstr. There are
bunch of other ways to break your code since noone saw it :)

With best regards, Anton Korobeynikov
Faculty of Mathematics and Mechanics, Saint Petersburg State University

More information about the llvm-commits mailing list