This is an archive of the discontinued LLVM Phabricator instance.

Use <cerrno> explicitly
ClosedPublic

Authored by chapuni on Apr 29 2023, 12:31 AM.

Details

Summary

D146395 has hidden errno in LLVM_ENABLE_MODULES.

Diff Detail

Event Timeline

chapuni created this revision.Apr 29 2023, 12:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 29 2023, 12:31 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
chapuni requested review of this revision.Apr 29 2023, 12:31 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 29 2023, 12:31 AM
chapuni added reviewers: philnik, Restricted Project.Apr 29 2023, 12:34 AM

Could you reconfirm they are expected?

ldionne accepted this revision.May 10 2023, 2:51 PM
ldionne added a subscriber: ldionne.

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 made the compiler catch the issue.

This revision is now accepted and ready to land.May 10 2023, 2:51 PM

@ldionne Thanks. It is what I would like to ask you.

This revision was landed with ongoing or failed builds.May 11 2023, 7:33 AM
This revision was automatically updated to reflect the committed changes.