[PATCH] D149513: Use <cerrno> explicitly
Louis Dionne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 10 14:51:46 PDT 2023
ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.
In D149513#4307343 <https://reviews.llvm.org/D149513#4307343>, @chapuni wrote:
> Could you reconfirm they are expected?
I don't understand the question. If these files use content from `<errno.h>`, they should include that or `<cerrno>`. I think you might be hitting a case of code not including what it uses previously, and D146395 <https://reviews.llvm.org/D146395> made the compiler catch the issue.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149513/new/
https://reviews.llvm.org/D149513
More information about the llvm-commits
mailing list