When a value is expected to be extended, we should emit an extended load rather than a normal G_LOAD.
Add checklines to arm64-abi.ll which show that we now emit the correct loads.
For ease of comparison: https://godbolt.org/z/8WvY6EfdE
Paths
| Differential D107313
[AArch64][GlobalISel] Emit extloads for ZExt/SExt values in assignValueToAddress ClosedPublic Authored by paquette on Aug 2 2021, 2:36 PM.
Details Summary When a value is expected to be extended, we should emit an extended load rather than a normal G_LOAD. Add checklines to arm64-abi.ll which show that we now emit the correct loads. For ease of comparison: https://godbolt.org/z/8WvY6EfdE
Diff Detail Event TimelineHerald added subscribers: danielkiss, hiraditya, kristof.beyls, rovka. · View Herald TranscriptAug 2 2021, 2:36 PM This revision is now accepted and ready to land.Aug 2 2021, 2:40 PM This revision was landed with ongoing or failed builds.Aug 2 2021, 2:50 PM Closed by commit rGbd13c8e610ca: [AArch64][GlobalISel] Emit extloads for ZExt/SExt values in assignValueToAddress (authored by paquette). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 363580 llvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp
llvm/test/CodeGen/AArch64/GlobalISel/call-lowering-signext.ll
llvm/test/CodeGen/AArch64/GlobalISel/call-lowering-zeroext.ll
llvm/test/CodeGen/AArch64/GlobalISel/call-translator-ios.ll
llvm/test/CodeGen/AArch64/arm64-abi.ll
|