[PATCH] D93187: [AMDGPU][NFC] Rename opsel/opsel_hi/neg_lo/neg_hi with suffix 0

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 13 17:09:39 PST 2020


critson created this revision.
critson added reviewers: rampitec, foad, Joe_Nash.
Herald added subscribers: kerbowa, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl, arsenm.
critson requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

These parameters set a default value of 0, so I believe they
should include a 0 suffix. This allows for versions which do not
set a default value in future.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93187

Files:
  llvm/lib/Target/AMDGPU/SIInstrInfo.td
  llvm/lib/Target/AMDGPU/VOP3Instructions.td
  llvm/lib/Target/AMDGPU/VOP3PInstructions.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93187.311477.patch
Type: text/x-patch
Size: 4703 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201214/3ed17c1d/attachment.bin>


More information about the llvm-commits mailing list