[PATCH] D30529: [GlobalISel] Enable specifying how to legalize non-power-of-2 size types.

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 11 09:46:00 PST 2017


rengolin added a comment.

Hi Kristof,

I haven't seen the patch at all, but what about situations where 64-bit is done with lib calls? For example, `ldivmod` takes 64-bit arguments and you wouldn't want to narrow them to 32-bits.

If this patch is intended to just simplify the legal vs. others, it shouldn't have a narrow-all that spans to +inf. Makes sense?

cheers,
--renato


https://reviews.llvm.org/D30529





More information about the llvm-commits mailing list