[PATCH] D74373: AMDGPU/GlobalISel: Handle sbfe/ubfe intrinsic

[Matt Arsenault via Phabricator via llvm-commits]

arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp:1484
+  auto OffsetMask = B.buildConstant(S32, maskTrailingOnes<unsigned>(16));
+  auto ClampOffset = B.buildAnd(S32, OffsetReg, OffsetMask);
+
----------------
foad wrote:
> Is the behaviour of the intrinsic well-defined when the "offset" argument is huge enough for this to make a difference?
I'm assuming the intrinsic is supposed to match the behavior of the V_BFE_* instructions, which ignore the high bits


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

https://reviews.llvm.org/D74373