[PATCH] D25624: Added 'inline' attribute to basic_string's destructor

Mehdi AMINI via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 28 18:28:08 PDT 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D25624#582752, @mehdi_amini wrote:

> (The commit message is confusing, it mentions "Currently basic_string's destructor is not getting inlined. So adding 'inline' attribute to ~basic_string()", please add the quote of the standard and mention that it enables instantiation)


(i.e. "inline" does not cause directly "inlining" here, this comes as a side effect of just starting to emit it)


https://reviews.llvm.org/D25624





More information about the cfe-commits mailing list