[PATCH] D148723: [clang] Enforce external linkage for inline builtin original declaration
serge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 2 09:07:27 PDT 2023
serge-sans-paille added a comment.
In D148723#4296112 <https://reviews.llvm.org/D148723#4296112>, @efriedma wrote:
> Really, I'd prefer to keep isInlineBuiltinDeclaration() targeted as narrowly as possible; part of that is making it not trigger for C++ inline functions (which it never did in the past).
I second that. Unfortunately the original bug showcase a situation that no longer holds the GNUinline attribute https://github.com/llvm/llvm-project/issues/61691
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148723/new/
https://reviews.llvm.org/D148723
More information about the cfe-commits
mailing list