[PATCH] D12087: always_inline codegen rewrite

Evgeniy Stepanov via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 3 17:23:02 PDT 2015


eugenis added a comment.

In http://reviews.llvm.org/D12087#239243, @yaron.keren wrote:

> In CGCXX.cpp, may be fixable after this commit:
>
>   // FIXME: An extern template instantiation will create functions with
>   // linkage "AvailableExternally". In libc++, some classes also define
>   // members with attribute "AlwaysInline" and expect no reference to
>   // be generated. It is desirable to reenable this optimisation after
>   // corresponding LLVM changes.


I've removed the checks for alwaysinline and available_externally. Tests + bootstrap do not show any new failures.


Repository:
  rL LLVM

http://reviews.llvm.org/D12087





More information about the cfe-commits mailing list