[Power9] Add missing instructions: extswsli, popcntb
ClosedPublic

Authored by stefanp on Thu, Aug 31, 9:57 AM.

Details

Summary

Added the following instructions:

  • extswsli
  • extswsli.
  • popcntb

Diff Detail

Repository
rL LLVM
stefanp created this revision.Thu, Aug 31, 9:57 AM
nemanjai added inline comments.Thu, Aug 31, 3:02 PM
lib/Target/PowerPC/PPCInstr64Bit.td
646 ↗(On Diff #113423)

This isn't the right instruction format to use as far as I can tell. This instruction *does not* modify CARRY according to the ISA.

stefanp updated this revision to Diff 113538.Fri, Sep 1, 6:55 AM

Fixed according to Nemanja's comment.

The update to the EXTSWSLI definition looks fine. However, you'll want to coordinate with https://reviews.llvm.org/D37404 for POPCNTB.

echristo accepted this revision.Tue, Sep 12, 3:49 PM

Let's go ahead with this one for now. Sam's patch can go on after if we decide to go with a C API way of accessing this rather than just inline assembly.

Thanks!

-eric

This revision is now accepted and ready to land.Tue, Sep 12, 3:49 PM
This revision was automatically updated to reflect the committed changes.