[PATCH] D131639: [OpenMP] Remove 'stdbool.h' from OpenMP header wrappers

Ivan Rodriguez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 09:40:48 PDT 2022


ivanrodriguez3753 added a comment.

This looks good to me, and I agree we should document what this is fixing. Any update on if/when this will land?

In my opinion, there's nothing broken about the user code (definitely contrived, though). They didn't ask for `stdbool.h` so there should not be a redeclaration error. This doesn't seem to be an issue when compiling for NVIDIA or host-only OpenMP, either.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131639/new/

https://reviews.llvm.org/D131639



More information about the cfe-commits mailing list