[PATCH] D107639: [AMDGPU][GISel] Smaller code for scalar 32 to 64-bit extensions

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 14 07:23:10 PST 2022


arsenm added a comment.

In D107639#3924314 <https://reviews.llvm.org/D107639#3924314>, @foad wrote:

> Do you still think this should be done as a post-regbankselect combine instead? I really have no idea.

I do think that could potentially enable other combines, but I'm not sure it needs to be exclusionary to also doing it in selection


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107639/new/

https://reviews.llvm.org/D107639



More information about the llvm-commits mailing list