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

Sanne Wouda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 8 07:35:33 PST 2017


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





More information about the llvm-commits mailing list