[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