[PATCH] RC->contains is a trap... (PR16106)

Jakob Stoklund Olesen stoklund at 2pi.dk
Mon Jun 17 10:09:27 PDT 2013


On Jun 12, 2013, at 4:36 PM, Eric Christopher <echristo at gmail.com> wrote:

> I'm fine with someone (possibly even me) making this change, but
> perhaps now we've decided to change the documented behavior of
> AltOrders? :) I'm definitely not against it since, to me, the
> definition of an AltOrder is an alternate ordering, not an entirely
> different register class. Probably should add an assert in tablegen to
> ensure that the N orderings all have the same size.

I think I would be OK with that. [TableGen should check that the sets have the same elements, not just the same number of elements, but that is what you meant].

Thanks,
/jakob




More information about the llvm-commits mailing list