[PATCH] [ARMv8] New NEON instructions vmaxnm/vminnm.

Tim Northover t.p.northover at gmail.com
Wed Jul 17 03:03:02 PDT 2013


Hi Joey,

(We discussed this on IRC, but just to recap for the list).

>   > 3. Why the "NEONDup" namespace? If shares a common feature with DUP,
>   > the namespace should probably be renamed to reflect that.
>
> The NEONDup is the only namespace that doesn't do pre-processing of
> instructions in the disassembler.

I think the NEONDup disassembly *does* add predicate operands, which
from what you said above is incorrect (though possibly harmless if
they're never accessed).

Because of that, we probably need a new namespace for them anyway, and
there's not much point in diverging from existing NEON behaviour of
having a single ARM/Thumb instruction.

Cheers.

Tim.



More information about the llvm-commits mailing list