[llvm] [NVPTX] Legalize aext-load to zext-load to expose more DAG combines (PR #154251)
Artem Belevich via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 20 11:23:13 PDT 2025
Artem-B wrote:
My hazy memory of MIPS is that signed/unsigned load only matters for byte/short variants `lb/lh`, so `lw/lwu` changes should be benign.
However, the changes that mention `# implicit-def: $a0_64` may need a closer look from someone familiar with MIPS. MIPS assembly and ABIs are funky, and those extra register moves may have been there for a reason. It's probably OK, too. AFAICT, the code just loads function arguments into a0/a1/a2 (AKA $4/$5/$6) before the function call. If anything the old code looks somewhat questionable, as $1 is typically reserved for the assembler use on MIPS.
https://github.com/llvm/llvm-project/pull/154251
More information about the llvm-commits
mailing list