aaron.ballman added a comment. In D140250#4077826 <https://reviews.llvm.org/D140250#4077826>, @iana wrote:
> In D140250#4076851 <https://reviews.llvm.org/D140250#4076851>, @aaron.ballman > wrote: > >> I'd expect there to be a test under `clang/test/Headers/` showing that this >> new header works as expected. > > It's covered pretty well by stddefneeds.cpp already. I can add a new one if > you want, but it'd pretty much be a copy/paste I was thinking of one similar to that but includes the new header and *not* stddef.h to demonstrate that this works entirely standalone. WDYT? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140250/new/ https://reviews.llvm.org/D140250 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
