[PATCH] D31856: Headers: Make the type of SIZE_MAX the same as size_t

[Duncan P. N. Exon Smith via Phabricator via cfe-commits]

dexonsmith added inline comments.


================
Comment at: clang/test/Headers/stdint-typeof-MINMAX.cpp:1
+// RUN: %clang_cc1 %s -ffreestanding -std=c++1z -fsyntax-only -triple=aarch64-none-none
+// RUN: %clang_cc1 %s -ffreestanding -std=c++1z -fsyntax-only -triple=arm-none-none
----------------
efriedma wrote:
> dexonsmith wrote:
> > I copied the `-ffreestanding` out of test/Preprocessor/stdint.c, but I'm wondering if it even serves a purpose here... if not, we could make the RUN lines shorter (and fit 80 columns without breaking in two).
> Without -ffreestanding, you're testing the contents of /usr/include/stdint.h rather than the compiler's builtin header.
Ah; I found no difference locally because I have no /usr/include.  `-v` shows me:

    ignoring nonexistent directory "/usr/include"

`-nostdsysteminc` has the same effect as `-ffreestanding`... but I guess I may as well leave it as is.


https://reviews.llvm.org/D31856