[PATCH] [AArch64] Add add/sub/cmp/cmn aliases to MC AsmParser

Jim Grosbach grosbach at apple.com
Fri Jul 11 12:52:40 PDT 2014


> On Jul 11, 2014, at 11:45 AM, Renato Golin <renato.golin at linaro.org> wrote:
> 
> On 11 July 2014 19:01, Jim Grosbach <grosbach at apple.com> wrote:
>> Hi Arnaud,
>> 
>> Does the ARM ARM specify these aliases? I can’t find them, if so, but it’s
>> not always the most readable document, so I could just be missing it.
> 
> Hi Jim,
> 
> Both ADD/ADDS and SUB/SUBS are implemented as AddWithCarry(), while
> CMN is an alias to ADDS and CMP is an alias to SUBS.
> 

That’s not what I’m asking, though. I’m asking where in the documentation it specifies that the assembler should accept aliases of these forms.

I have the same question about a number of the other proposed patches of this nature. If the docs say we should support them, then I’m all for it. If they don’t, I’m very hesitant to do so. I don’t like adding undocumented extensions to the assembler syntax.

-Jim





More information about the llvm-commits mailing list