[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