[PATCH] D137932: [GlobalIsel][AMDGPU] Changing legalize rule for G_{UADDO|UADDE|USUBO|USUBE|SADDE|SSUBE}

Petar Avramovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 14 03:52:39 PST 2022


Petar.Avramovic added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/GlobalISel/legalize-sadde.mir:2
 # NOTE: Assertions have been autogenerated by utils/update_mir_test_checks.py
 # RUN: llc -mtriple=amdgcn-mesa-mesa3d -mcpu=fiji -O0 -run-pass=legalizer -global-isel-abort=0 %s -o - | FileCheck %s
 
----------------
I think that -global-isel-abort=0 in no longer required since test should compile now with the addition of narrow scalar of s64 to s32.
Can you check the other tests?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137932



More information about the llvm-commits mailing list