[PATCH] D55427: [libcxx] Call __count_bool_true for bitset count

Renato Golin via Phabricator reviews at reviews.llvm.org
Fri Dec 14 03:16:08 PST 2018


rengolin added subscribers: peter.smith, rengolin.
rengolin added a comment.

Getting a bit late in this discussion, as we had an internal one just recently.

The change to remove always_inline in a number of libc++ template functions is a good one, especially when the inliner can guess and does a good job already.

In this case, however, because the type is a reference, the inliner would require a lot more effort to inspect the uses (and side-effects).

Improving the inliner here would be a huge hammer, probably increasing compile time for all codes for the minimal benefit of this very special case.

Then perhaps, it would be beneficial and pragmatic, to revert that removal in this special case.

Makes sense?

cheers,
--renato


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

https://reviews.llvm.org/D55427





More information about the libcxx-commits mailing list