aaron.ballman added inline comments.
================ Comment at: clang-tools-extra/test/clang-tidy/checkers/Inputs/Headers/stddef.h:12 + +using size_t = long long unsigned; + ---------------- barcisz wrote: > 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 `using size_t = decltype(sizeof(0));` is a better way to spell this, and I'd probably inline it rather than add a new header file. 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
