[PATCH] D134201: [Sparc] Fix useDeprecatedPositionallyEncodedOperands errors.

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 26 08:26:25 PDT 2022


jyknight added a comment.

In D134201#3883934 <https://reviews.llvm.org/D134201#3883934>, @MaskRay wrote:

> LDSTUBrr/LDSTUBri have differences as mentioned. Is the    case 74: difference expected?

Yep, that's expected (the decoder currently just silently ignores non-matching operands, and "dst" didn't match anything, but "rd" does -- so now it emits (correctly) an additional operand in decoding). It's still broken, as mentioned, though, since the sub-operands aren't supported in decoding yet. Patch coming soon will address that and make other improvements here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134201



More information about the llvm-commits mailing list