[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