[PATCH] GlobalISel: Implement narrowing for G_STORE

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 09:44:21 PST 2017


Tim Northover <t.p.northover at gmail.com> writes:
> Hi Justin,
>
> On 17 January 2017 at 17:01, Justin Bogner via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> This implements narrowing of stores in the obvious way, but it looks a
>> little verbose to me with all of the extra vregs for the constants and
>> the geps. WDYT?
>
> I think this looks fine. We don't change the MachineMemOperand, but I
> think that's conservatively correct so we can worry about it in the
> unlikely event that it ever matters.

r292412. Thanks Tim!


More information about the llvm-commits mailing list