aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM with a small change to the test coverage. ================ Comment at: clang/test/Headers/limits.cpp:8 +// Specifically test 16-bit int platforms. +// RUN: %clang_cc1 -triple=avr -ffreestanding -fsyntax-only -verify -x c %s + ---------------- Might as well add a C++ RUN as well to make sure we cover both language modes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144218/new/ https://reviews.llvm.org/D144218 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
