[PATCH] D30571: [ARM] [Assembler] Extend immediate conversions for A32, T32 and T16

[Sanne Wouda via Phabricator via llvm-commits]

sanwou01 added a comment.

Hi Renato,

Thank you for the references. I hadn't looked at Aarch64 yet, but looking through those patches quickly, it looks like all conversions for negative immediates are already supported.  It looks like it is using the same aliasing mechanism that I'm using.  Changing Aarch64 to use InstSubst instead of InstAlias should make sure that -mattr=+no-imm-conversions works for Aarch64 too.

That said, maybe NegativeImmediates/no-neg-immediates is an even better name for this feature!

Thanks,
Sanne


https://reviews.llvm.org/D30571