[PATCH] D107684: [NFC][AVR][clang] Add test for D107672

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 11 10:27:45 PDT 2021


MaskRay added a comment.

In D107684#2936034 <https://reviews.llvm.org/D107684#2936034>, @mhjacobson wrote:

> In D107684#2936033 <https://reviews.llvm.org/D107684#2936033>, @MaskRay wrote:
>
>> `--sysroot %S/Inputs/basic_avr_tree/usr/lib` seems incorrect.
>>
>> sysroot is usually a directory which contains `usr/lib` and `lib/`. `somewhere/usr/lib` is not usually used as a sysroot.
>
> Yeah, I agree it's weird.  This is basically a shortcut to avoid creating a duplicate sysroot in `Inputs`.  Would you prefer I create the duplicate tree?

A duplicate tree is better. Otherwise you can use `unittests/Driver/ToolChainTest.cpp`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107684



More information about the cfe-commits mailing list