mhjacobson added a comment.

In D107682#2932685 <https://reviews.llvm.org/D107682#2932685>, @benshi001 wrote:

> In D107682#2932607 <https://reviews.llvm.org/D107682#2932607>, @mhjacobson 
> wrote:
>
>> For other examples of using four `..`s, look in `Gnu.cpp`, where there are 
>> several places where `getParentLibPath()` is appended with `/../`.  
>> `AddMultilibPaths()`, `PushPPaths()`.
>>
>> `getParentLibPath()`, in turn, is `GCCInstallPath/../../../`
>
> Thanks for your help. I have updated my patch. Now it should work for both 
> your platform and mine. It search `GCCRoot.getParentLibPath()/avr` first, 
> otherwise `GCCRoot.getParentLibPath()/../avr`.
>
> Please help me check.

Latest diff works for me, thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107682/new/

https://reviews.llvm.org/D107682

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to