barcisz added inline comments.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/Inputs/Headers/stddef.h:12 + +using size_t = long long unsigned; + ---------------- njames93 wrote: > If this is all the file is used for, and it's only used for this one test > file, can probably remove this header and inline the definition. > I also feel this definition may be fragile on certain platforms. I wanted to add it in a separate header for future proofness; as for the definition, I cannot really use uint64_t here since I cannot include the real stddef.h Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133436/new/ https://reviews.llvm.org/D133436 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
