[PATCH] D92156: [PowerPC] Add support for "tlbiel" with two arguments

[Jinsong Ji via Phabricator via llvm-commits]

jsji added a comment.

I don't know where do you get the syntax of tlbiel with 2 operands.

According to ISA 3.1 https://ibm.ent.box.com/s/hhjfw0x0lrbtyzmiaffnbxh2fuo0fog0 Page 1239
there is only two forms, one with 5 operands, one with one operands.

  tlbiel serves as both a basic and an extended mne-monic.  The Assembler will recognize a tlbiel mne-monic with five operands as the basic form, and a tlbiel mnemonic with one operand as the extended form.  In the extended form the RS, RIC, PRS, andR operands are omitted and assumed to be 0.

So maybe we should add the 5 operands basic form here.

Also, looks like there is bug for `tlbie` as well, there is NO one operand mnemonic, only 5 or 2.

  tlbie serves as both a basic and an extended mne-monic.  The Assembler will recognize a tlbie mne-monic with five operands as the basic form, and a tlbie mnemonic with two operands as the extended form.    In  the  extended  form  the  RIC,  PRS,  and  Roperands are omitted and assumed to be 0.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92156