[all-commits] [llvm/llvm-project] 922b7b: [Clang][OpenMP] Fix stdio.h wrapper when glibc inc...

Joel E. Denny via All-commits all-commits at lists.llvm.org
Fri Jan 5 07:22:23 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 922b7b8bf465ddc292fa91bd6a860510a1eea6e2
      https://github.com/llvm/llvm-project/commit/922b7b8bf465ddc292fa91bd6a860510a1eea6e2
  Author: Joel E. Denny <jdenny.ornl at gmail.com>
  Date:   2024-01-05 (Fri, 05 Jan 2024)

  Changed paths:
    M clang/lib/Headers/llvm_libc_wrappers/stdio.h

  Log Message:
  -----------
  [Clang][OpenMP] Fix stdio.h wrapper when glibc includes again (#77017)

Since D154036 landed (2a65d0388ca0 on July 7, 2023), I've been seeing
many (40-50) libomptarget test failures with errors like the following
on some of our test systems:

```
/auto/software/gcc/x86_64/gcc-11.1.0/lib/gcc/x86_64-pc-linux-gnu/11.1.0/../../../../include/c++/11.1.0/cstdio:99:11: error: no member named 'fpos_t' in the global namespace
   99 |   using ::fpos_t;
      |         ~~^
```

This patch fixes that and doesn't break our other test sytems. I've
looked through the glibc history and at LLVM's libc stdio.h to give me
confidence that this solution should work on other systems. Of course,
there might be use cases I've overlooked, so feedback is appreciated.




More information about the All-commits mailing list