[all-commits] [llvm/llvm-project] e38158: [libc] Work around lack of '__has_builtin' for GPU...

Joseph Huber via All-commits all-commits at lists.llvm.org
Tue Apr 2 15:06:26 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e381586f259568bf244fcd857ce91fc5cb38b959
      https://github.com/llvm/llvm-project/commit/e381586f259568bf244fcd857ce91fc5cb38b959
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2024-04-02 (Tue, 02 Apr 2024)

  Changed paths:
    M libc/src/__support/macros/config.h

  Log Message:
  -----------
  [libc] Work around lack of '__has_builtin' for GPU server (#87417)

Summary:
The RPC server build for the GPU support needs to be build from the
"projects" phase of the LLVM build. That means it is built with the same
compile that LLVM supports, which currently is GCC 7.4 in most cases.
A previous patch removed the `LIBC_HAS_BUILTIN` indirection we used,
which regressed the case where we used the `libc` source externally. The
files that we need to use here are `converter.cpp` and `writer.cpp`
which currently are compatible with C++17, so there aren't issues with
the code itself. However, older GCC does not have this builtin which
makes the checks fail.

This patch just adds in a simple wrapper that allows it to correctly
ignore everything if using a compiler that doesn't support it.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list