[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