[PATCH] D157547: Arm64EC entry/exit thunks, consolidated.

Jacek Caban via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 9 10:25:05 PDT 2023


jacek added a comment.

It works nice with my testing so far, thanks!

I noticed one problem with code that defines symbols in assembly. It conflicts with anti-dependency symbols emitted by codegen and hits an asserts in assembly printer. This commit fixes it for me:
https://github.com/cjacek/llvm-project/commit/879b1b6bf99b1e506ae04b8db0b44eb1b2205f19

Here is a test case:

  extern void asmfunc(void);
  __asm__( ".globl \"#asmfunc\"\n"
           "\t.section .text,\"xr\",discard,\"#asmfunc\"\n"
           "\"#asmfunc\":\n\t"
           "nop; ret\n\t" );
  void test(void) { asmfunc(); }


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157547/new/

https://reviews.llvm.org/D157547



More information about the cfe-commits mailing list