[PATCH] replace some const std::string & with llvm::StringRef or std::string

[Yaron Keren]

Hi Eugene,

This patch does not pass clang extra tools regression tests locally,

FAIL: Clang Tools :: modularize/NoProblems.modularize (7141 of 19860)
Testing: 0 .. 10.. 20.. 30
FAIL: Clang Tools :: modularize/NoProblemsDependencies.modularize (7144 of
19860)
Testing: 0 .. 10.. 20.. 30
FAIL: Clang Tools :: modularize/NoProblemsCoverage.modularize (7145 of
19860)
(...)

Please check it out.

Yaron


2015-06-22 15:25 GMT+03:00 Eugene Kosov <claprix at yandex.ru>:

> Hi.
>
> Here are two patches for clang-tools-extra and for clang which simplifies
> code a little bit and also removes possibility of implicit std::string
> construction.
>
> --
> Eugene
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150626/1bf72d93/attachment.html>