[llvm] [DAGCombiner][AMDGPU] Track signedness in ByteProviders (PR #65995)
Jeffrey Byrnes via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 12 08:25:09 PDT 2023
================
@@ -10558,11 +10562,15 @@ calculateSrcByte(const SDValue Op, uint64_t DestByte, uint64_t SrcIndex = 0,
SrcIndex += BitShift / 8;
- return calculateSrcByte(Op->getOperand(0), DestByte, SrcIndex, Depth + 1);
+ return calculateSrcByte(Op->getOperand(0), DestByte, SrcIndex, IsSigned,
+ Depth + 1);
}
default: {
- return ByteProvider<SDValue>::getSrc(Op, DestByte, SrcIndex);
+ if (auto L = dyn_cast<LoadSDNode>(Op))
----------------
jrbyrnes wrote:
Yes you are right --
Even so, I propose to blacklist for now.
https://github.com/llvm/llvm-project/pull/65995
More information about the llvm-commits
mailing list