[PATCH] D129572: [X86] initial -mfunction-return=thunk-extern support

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 12 21:32:48 PDT 2022


nickdesaulniers added inline comments.


================
Comment at: clang/test/CodeGen/attr-function-return.c:42
+// CHECK: @double_keep_thunk2() [[EXTERN]]
+[[gnu::function_return("thunk-keep")]][[gnu::function_return("thunk-extern")]]
+void double_keep_thunk2(void) {}
----------------
I just noticed this typo, from an earlier version that did implement `thunk-keep` (which isn't useful ATM).  So I should fix this up; nothing is being merged here.

I need to `s/thunk-//`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129572



More information about the cfe-commits mailing list