[libc-commits] [libc] [libc] Fix missing Annex K declarations caused by header include order (PR #229587)

[Chen Miao via libc-commits]

ChenMiaoi wrote:

> Since Annex K support is still early, this is one patch that requires a local build and test by the reviewer, as much is not yet covered by regression tests. It might take me some time to get around to validate this.
> 
> This was supposed to be a bug for LLVM beginners, hence AI should have been discouraged. But I forgot to tag the issue as such. It's not your fault. But try not to use AI for your initial work in LLVM. It may defeat the whole point of learning.



> Since Annex K support is still early, this is one patch that requires a local build and test by the reviewer, as much is not yet covered by regression tests. It might take me some time to get around to validate this.
> 
> This was supposed to be a bug for LLVM beginners, hence AI should have been discouraged. But I forgot to tag the issue as such. It's not your fault. But try not to use AI for your initial work in LLVM. It may defeat the whole point of learning.

Thanks for clarifying. I understand the reasoning behind keeping good first issues focused on learning, and I’ll keep that in mind.

https://github.com/llvm/llvm-project/pull/229587