[PATCH] D66963: [PowerPC] Support extended mnemonics mffprwz etc.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 14:22:44 PDT 2019


hfinkel accepted this revision as: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

In D66963#1651566 <https://reviews.llvm.org/D66963#1651566>, @jsji wrote:

> In D66963#1651477 <https://reviews.llvm.org/D66963#1651477>, @hfinkel wrote:
>
> > > mfvrwz       RA,VRS    is   mfvsrwz      RA,VRS+32
> >
> > The '+32' bit is covered by the added CodeGen/PowerPC/inlineasm-extendedmne.ll test, but I'd really like to see direct encoding tests for all of the instructions. Can you please make sure that they all appear in MC/PowerPC/vsx.s and MC/Disassembler/PowerPC/vsx.txt?
>
>
> Good point, and updated.


LGTM


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66963





More information about the llvm-commits mailing list