[PATCH] D43788: Merge {COFF, ELF}/Strings.cpp to Common/Strings.cpp.

[Rui Ueyama via llvm-commits]

I'm sorry but I don't think that the issue you found is critical enough to
make a last-minute change. It's a nice-to-have fix, but LLVM 6.0 was
supposed to be release last week, so...


On Thu, Mar 1, 2018 at 3:48 AM Hagai Cohen <hagai.co at gmail.com> wrote:

> Hi,
>
> Ive seen that release 6 hasn’t officially released yet (and planned for
> mid-march),
> Will this change get into release 60 branch?
> Or we will have to wait till release 7?
> On Wed, 28 Feb 2018 at 19:41 Rui Ueyama via Phabricator <
> reviews at reviews.llvm.org> wrote:
>
>> This revision was automatically updated to reflect the committed changes.
>> Closed by commit rL326339: Merge {COFF,ELF}/Strings.cpp to
>> Common/Strings.cpp. (authored by ruiu, committed by ).
>>
>> Changed prior to commit:
>>   https://reviews.llvm.org/D43788?vs=135991&id=136317#toc
>>
>> Repository:
>>   rL LLVM
>>
>> https://reviews.llvm.org/D43788
>>
>> Files:
>>   lld/trunk/COFF/CMakeLists.txt
>>   lld/trunk/COFF/Strings.cpp
>>   lld/trunk/COFF/Strings.h
>>   lld/trunk/COFF/Symbols.cpp
>>   lld/trunk/Common/Strings.cpp
>>   lld/trunk/ELF/AArch64ErrataFix.cpp
>>   lld/trunk/ELF/CMakeLists.txt
>>   lld/trunk/ELF/Driver.cpp
>>   lld/trunk/ELF/EhFrame.cpp
>>   lld/trunk/ELF/LinkerScript.cpp
>>   lld/trunk/ELF/LinkerScript.h
>>   lld/trunk/ELF/MapFile.cpp
>>   lld/trunk/ELF/MarkLive.cpp
>>   lld/trunk/ELF/OutputSections.cpp
>>   lld/trunk/ELF/Relocations.cpp
>>   lld/trunk/ELF/Strings.cpp
>>   lld/trunk/ELF/Strings.h
>>   lld/trunk/ELF/SymbolTable.h
>>   lld/trunk/ELF/Symbols.h
>>   lld/trunk/ELF/SyntheticSections.cpp
>>   lld/trunk/ELF/Writer.cpp
>>   lld/trunk/include/lld/Common/Strings.h
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180301/d6d5b953/attachment.html>