[PATCH] D97669: [clang][AVR] Add avr-libc/include to clang system include paths
Ben Shi via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 8 19:48:32 PDT 2021
benshi001 added a comment.
In D97669#2676826 <https://reviews.llvm.org/D97669#2676826>, @Anastasia wrote:
> In D97669#2665865 <https://reviews.llvm.org/D97669#2665865>, @benshi001 wrote:
>
>> In D97669#2661560 <https://reviews.llvm.org/D97669#2661560>, @Anastasia wrote:
>>
>>> Is `stdio.h` used by anything?
>>
>> No. `stdio.h` is not used. But if I do `#include <avr/interrupt.h>`, then `-I /usr/lib/avr/include` must be specified in the command line option. While avr-gcc does not reqiures that.
>>
>> I would like to keep clang & avr-gcc in the behaviour.
>
> Ok, do you plan to use it later? Then perhaps you should be adding it in the subsequent patches?
No. No future pathes are needed. The avr-libc includes standard libc (headers and libs) and avr specific headers and libs. This patch fixes all of these issues. Both standard libc and avr platform specific libs can be used without any explict command line option, just like avr-gcc does.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97669/new/
https://reviews.llvm.org/D97669
More information about the cfe-commits
mailing list