benshi001 marked an inline comment as done. benshi001 added inline comments.
================ Comment at: clang/lib/Basic/Targets/AVR.cpp:380 Builder.defineMacro("__AVR"); + Builder.defineMacro("__avr"); Builder.defineMacro("__AVR__"); ---------------- MaskRay wrote: > The clang preprocessor changes are unrelated to compiler-rt and should be > contribute separated with tests. I have created another patch as you suggested, thanks. https://reviews.llvm.org/D123567 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123200/new/ https://reviews.llvm.org/D123200 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits