[PATCH] D83825: AMDGPU: Rename add/sub with carry out instructions

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 02:02:49 PDT 2020


foad added a comment.
Herald added a subscriber: wuzish.

Sounds great to me, but I'm not much of an expert on instruction definitions.



================
Comment at: llvm/lib/Target/AMDGPU/VOP2Instructions.td:1275
+
+// V_ADD_I32, V_SUB_I32, and V_SUBREV_I32 where renamed to *_U32 in
+// VI, but the VI instructions behave the same as the SI versions.
----------------
"were"


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

https://reviews.llvm.org/D83825





More information about the llvm-commits mailing list