[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