AaronBallman wrote:

> > Release note and test coverage?
> 
> I tested it locally. I copied code to `prebuilt/.../limits.h` in aosp and 
> removed bionic's copy and rebuilt to see whether it works. And it worked 
> well. Any extra upstream test coverage needed?

Yes, we've got tests for Clang-supplied headers in `clang/test/Headers`; you 
should pass `-ffreestanding` on the `RUN` line and that should find Clang's 
headers rather than the system headers.

https://github.com/llvm/llvm-project/pull/115406
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to