'.inst' in inline-assembly changed to '.insn'

[CyoeeA9e]

The correct directive name should be .insn. Here is the issue in rust repository.

[ehuss]

I'm a bit confused, as I believe .inst is a valid directive still. What is the difference between them? Is there some more information about this change?

@Amanieu Can you provide some information here?

[Amanieu]

Both are correct: .inst is used on ARM, .insn is used in RISC-V.

[ehuss]

@joshtriplett Do you want to add .insn to the guaranteed directive list? (Not what this PR does, it should be added not replaced.)

Looks like .insn was added to gnu-as in 2018 in bminor/binutils-gdb@0e35537 and in llvm in llvm-14 in llvm/llvm-project@2838797 (I think, from some quick googling).

[Amanieu]

I'm happy to add it in addition to .inst.

[joshtriplett]

@ehuss Yeah, that seems reasonable.