[PATCH] D22782: Added 'inline' attribute to __init to inline the basic_string's constructor

Aditya Kumar via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 09:39:38 PDT 2016


hiraditya added a comment.

In https://reviews.llvm.org/D22782#495436, @mclow.lists wrote:

> Do we have a test for the problem that this is solving?


We are looking at the libcxx testsuite, it seems there are only correctness/functionality tests. Do you have any pointers on how to add tests to check if functions are inlined. In llvm it is easy to do because we can print the assembly and use FileCheck/grep to CHECK/CHECK-NOT if a string is present.


https://reviews.llvm.org/D22782





More information about the cfe-commits mailing list