Excerpts from Christophe Leroy's message of March 23, 2021 2:49 am: > > > Le 19/03/2021 à 03:01, Nicholas Piggin a écrit : >> Excerpts from Daniel Axtens's message of February 25, 2021 1:10 pm: >>> The llvm integrated assembler does not recognise the ISA 2.05 tlbiel >>> version. Eventually do this more smartly. >> >> The whole thing with TLBIE and TLBIEL in this file seems a bit too >> clever. We should have PPC_TLBIE* macros for all of them. > > I was expecting to drop PPC_* macros as much as possible taking into account > the later binutils > support most of them (https://github.com/linuxppc/issues/issues/350). Was not > expecting to go the > other direction.
The problem in this file is we generate 3 different tlbie and tlbiel instructions with the same mnemonic corresponding to different ISA versions. This might actually be the one good place to use .machine to make sure the assembler generates the right thing. I'm not entirely sure it is foolproof because some of the times the instruction variant is inferred by the number of arguments it has yet arguments can be implicit. PPC_ define would be exactly explicit. But if it can be made reasonably robust with .machine then I'd be okay with that too. Thanks, Nick
