[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
Mon May 17 19:30:16 PDT 2021
benshi001 added a reviewer: efriedma.
benshi001 added a subscriber: efriedma.
benshi001 added a comment.
@efriedma
Could you please help me review this patch? It relates to the one you have reviewed for me, and is also about wrong mangled function name in c++ on AVR.
The previous patch has correctly handled __UINT8/16/32/64_TYPE__,but current clang-avr still be wrong at int8/16/32/64_t, the underlying cause is the system's stdint.h is used first, which it should be avr-libc's stdint.h going first. (avr-g++ does so)
The patch fixes that issue, which add avr-libc to clang-avr toolchain's include path.
Thanks for your help!!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97669/new/
https://reviews.llvm.org/D97669
More information about the cfe-commits
mailing list