[PATCH] GlobalISel: Implement narrowing for G_STORE

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 09:00:57 PST 2017


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.

Cheers.

Tim.


More information about the llvm-commits mailing list