Change const std::string & to StringRef in Lexer

[Rafael Espíndola]

For the first few patches you just ask someone to commit it.

In this case, two things I noticed when I applied the patch:

* It is for clang, so you should send it to cfe-commits.
* It is incomplete. Looks like you only included the header changes:


llvm/tools/clang/lib/Lex/PTHLexer.cpp:434:25: error: out-of-line
definition of 'Create' does not match any declaration in
'clang::PTHManager'
PTHManager *PTHManager::Create(const std::string &file,


On 1 June 2015 at 02:27, Косов Евгений <claprix at yandex.ru> wrote:
> Cool. Thanks for reviewing. How now can I commit this patch into trunk?
>
> Eugene
>
> 01.06.2015, 03:12, "Rafael Espíndola" <rafael.espindola at gmail.com>:
>> LGTM
>>
>> On 31 May 2015 at 15:56, Косов Евгений <claprix at yandex.ru> wrote:
>>>  Hi.
>>>
>>>  This is a small and boring patch. I'm checking a patch submitting process. Does it make sense to change more const string references in other part of LLVM?
>>>
>>>  Eugene
>>>  _______________________________________________
>>>  llvm-commits mailing list
>>>  llvm-commits at cs.uiuc.edu
>>>  http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits