[PATCH] D22008: GlobalISel: implement low-level type suitable for MachineInstr selection

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 11:23:26 PDT 2016


> On Jul 6, 2016, at 11:21 AM, Tim Northover <t.p.northover at gmail.com> wrote:
> 
>> The bottom line is, this is for a smoother transition from SDISel to GISel than actually needed for GISel.
> 
> I can certainly live with it as a temporary hack like that, I'll start
> adding it. But I think we should be *very* careful about extending its
> use or we will just end up with a messy 2-location type system.

Agreed.

As for now, I would say to live it out and we will see if we need to resurrect it when adding a new backend.

Q.

> 
> Tim.



More information about the llvm-commits mailing list