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

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 10:26:38 PDT 2022


jhuber6 added a comment.

In D131639#3749563 <https://reviews.llvm.org/D131639#3749563>, @ivanrodriguez3753 wrote:

> The user didn't define any `__` or `_[A-Z]` identifiers, though?

Am I misunderstanding the test input?

  /* Visual Studio < 2013 does not have stdbool.h so here it is a replacement: */
  #if defined __STDC__ && defined __STDC_VERSION__ && __STDC_VERSION__ >= 199901L
  /* have a C99 compiler */
  typedef _Bool bool;
  #else
  /* do not have a C99 compiler */
  typedef unsigned char bool;
  #endif

I assumed this was the user defining this on their own.


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