[PATCH] D115387: [instcombine] Canonicalize constant index type to i64 for extractelement/insertelement
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 9 10:35:30 PST 2021
reames added a comment.
In D115387#3183282 <https://reviews.llvm.org/D115387#3183282>, @craig.topper wrote:
> In D115387#3183235 <https://reviews.llvm.org/D115387#3183235>, @reames wrote:
>
>> Sounds like the consensus is i32? I'll give it another day for further discussion, and will then update the patch if that's the direction we want to go in.
>
> Would we update IRBuilder to use i32 and perhaps change the interface from uint64_t to unsigned?
Can we not increase the scope of the patch? This is purely a minor cleanup as far as I'm concerned, and if it grows too much, I'll simply abandon it as not worth the effort.
I am particularly concerns about changing argument types of the public API given possibility for silent downcast.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115387/new/
https://reviews.llvm.org/D115387
More information about the llvm-commits
mailing list