[PATCH] D70758: [PowerPC][NFC] Rename record instructions to use r suffix instead of o

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

Yeah, I actually added my initial comment precisely because I thought the existing form is more familiar to those that do not have deep experience with the ISA. What I mean is that the `r` is fairly intuitive to those of us that are deeply familiar with the ISA - it means "record-form". But to those less familiar with the ISA, the phrase "record-form" may be completely foreign and the choice of `r` would then be quite confusing. Whereas looking at the mnemonics in the ISA:

  add
  add.
  addo
  addo.

and the corresponding mnemonics in our td files:

  ADD
  ADDo
  ADDO
  ADDOo

it does not require a lot of familiarity to connect the uppercase letters to the letters in the mnemonic and the lowercase letter to the dot.

(note that I do realize that these have 4/8 on them and chose those to illustrate that the more confusing part to a newcomer is the 32/64 bit versions)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70758