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

Tue Jul 12 11:12:39 PDT 2022

nickdesaulniers added a comment.

In D129572#3645934 <https://reviews.llvm.org/D129572#3645934>, @erichkeane wrote:

> Our typical rule is to keep the 1st one I think, and reject the 2nd.

But then the _codegen_ will differ from GCC.  And we _want_ clang to be a drop in replacement, so differing in behavior there is unacceptable.

https://godbolt.org/z/rf16T83Kj

IMO, the standards bodies focusing on standardizing attributes should clarify the semantics of attribute merging, _then_ compiler vendors should fix their compilers.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129572