<div dir="rtl"><div dir="ltr">OK, done.</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2013/10/21 Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span></div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sat, Oct 19, 2013 at 1:07 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>


> Hi Hans,<br>
><br>
> I just got commit access after approval rights.<br>
><br>
> Is it OK to commit this patch ?<br>
<br>
</div>Looks good to me, but I agree with David's comment about doing<br>
"MainFileName = MainFileDirSS.str()" - it's easier on the eyes.<br>
<br>
Thanks,<br>
Hans<br>
<div class="HOEnZb"><div class="h5"><br>
> 2013/10/17 Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>><br>
>><br>
>> On Thu, Oct 17, 2013 at 8:09 AM, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>><br>
>> wrote:<br>
>> > Hi,<br>
>> ><br>
>> > Filename appending in CGDebugInfo.cpp was implemented as simple string<br>
>> > concatenation, resulting in duplicate file debug information on Windows<br>
>> > due<br>
>> > to wrong path separator.<br>
>> ><br>
>> > The patch uses  llvm::sys::path::append to properly append the filename<br>
>> > on<br>
>> > all OS.<br>
>> ><br>
>> > Is there a way to use path::append without the SmallString temporary?<br>
>> ><br>
>> > Yaron<br>
>><br>
>> The patch looks reasonable to me, but I wish there were a test we<br>
>> could update for this.<br>
>><br>
>> Maybe the debug info folks know if there is a test that picks up the<br>
>> -main-file-name?<br>
>><br>
>>  - Hans<br>
><br>
><br>
</div></div></blockquote></div><br></div>