[PATCH] D94795: [AMDGPU] Fix use of HasModifiers in VopProfile

Joe Nash via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 12:02:38 PST 2021


Joe_Nash accepted this revision.
Joe_Nash added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.td:2071
+                               isModifierType<Src2VT>.ret,
+                               HasOMod,
+                               EnableF32SrcMods);
----------------
It would be even more consistent if HasClamp was here too, so that HasModifiers is true if HasClamp is true. However, I think that can be a separate patch. 


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

https://reviews.llvm.org/D94795



More information about the llvm-commits mailing list