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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to