[PATCH] D107682: [AVR][clang] Improve search for avr-libc installation path

Matt Jacobson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 9 16:07:52 PDT 2021


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



More information about the cfe-commits mailing list