public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Akira Moroo" <retrage01@gmail.com>
To: devel@edk2.groups.io, pedro.falcato@gmail.com,
	Andrew Fish <afish@apple.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Subject: Re: [edk2-devel] AArch64 assembly compatibility for Xcode and GCC
Date: Thu, 6 Jan 2022 19:38:31 +0900	[thread overview]
Message-ID: <554BF0E3-656B-4FA7-A6E3-509344023851@gmail.com> (raw)
In-Reply-To: <CAKbZUD3B=Uajs3Z+STNuPVuA5mutn_52izM3QD0EtL=33B27Lw@mail.gmail.com>

Hi,

I tried to add support AArch64 XCODE5 about three months ago, too. I had the same compatibility issue in the other code. I just ended up adding ugly #ifdef as a workaround [1]. I think there are other compatibility issues in the current EDK2 code base. I’m not sure if the GAS macro and Apple Clang Assembler are compatible, but the C standard guarantees C preprocessor macro compatibility.

[1] https://github.com/retrage/edk2/commit/89b166758b9f858373bb20eb8e006914f7f13373

Best regards,

Akira Moroo

> On Jan 6, 2022, at 8:31, Pedro Falcato <pedro.falcato@gmail.com> wrote:
> 
> Hi,
> 
> GNU Assembler and the clang assembler support more complex macros natively (using the .macro and.endm directives), without the need for the C preprocessor.
> This completely sidesteps the need for explicit ";" as the GAS macros respect newlines inside the macro.
> 
> See https://godbolt.org/z/6zzqoeKE4 for a very simple example.
> Of course, you can also pass arguments, see https://sourceware.org/binutils/docs/as/Macro.html for the full documentation.
> 
> Best regards,
> Pedro
> 
> On Wed, Jan 5, 2022 at 8:36 PM Andrew Fish via groups.io <afish=apple.com@groups.io> wrote:
> I was playing around with getting Xcode to compile AArch64 code and I hit an interesting compatibility issue. The assembler line continuation token for GCC (;) is a comment for the Xcode clang assembler. For Xcode clang you need to use %%. Yikes [1]
> 
> I was wondering if any one else has hit this in another project and if there was any cool workaround?
> 
> The only thing I can think of is the “big hammer” on both (well you only need one) ends. We could replace ; with a #define that matches the assembler. It looks like this issue mostly hits in Code that is using C pre-processor macros so we could refactor the code. I’m a little concerned that the C pre processor is getting used since the macro languages of the assembler might also have some compatibility issues?
> 
> Looking for ideas and opinions on the best way to fix this if we want to add Xcode as an ARM compiler in the future. 
> 
> 
> [1] https://github.com/tianocore/edk2/blob/master/MdePkg/Library/BaseLib/AArch64/SetJumpLongJump.S#L13
> 
> #define GPR_LAYOUT                         \
>         REG_PAIR (x19, x20,  0);           \
>         REG_PAIR (x21, x22, 16);           \
>         REG_PAIR (x23, x24, 32);           \
>         REG_PAIR (x25, x26, 48);           \
>         REG_PAIR (x27, x28, 64);           \
>         REG_PAIR (x29, x30, 80);/*FP, LR*/ \
>         REG_ONE  (x16,      96) /*IP0*/
> 
> I had to change it to:
> 
> #define GPR_LAYOUT                         \
>         REG_PAIR (x19, x20,  0)%%          \
>         REG_PAIR (x21, x22, 16)%%          \
>         REG_PAIR (x23, x24, 32)%%          \
>         REG_PAIR (x25, x26, 48)%%          \
>         REG_PAIR (x27, x28, 64)%%          \
>         REG_PAIR (x29, x30, 80)%%/*FP, LR*/ \
>         REG_ONE  (x16,      96) /*IP0*/
> 
> Thanks,
> 
> Andrew Fish
> 
> 
> 
> 
> 
> -- 
> Pedro Falcato
> 


      reply	other threads:[~2022-01-06 10:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-05 20:36 AArch64 assembly compatibility for Xcode and GCC Andrew Fish
2022-01-05 23:31 ` [edk2-devel] " Pedro Falcato
2022-01-06 10:38   ` Akira Moroo [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=554BF0E3-656B-4FA7-A6E3-509344023851@gmail.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox