[all-commits] [llvm/llvm-project] 483db5: compiler-rt: Allow build without __c11_atomic_fetc...

Sergei Trofimovich via All-commits all-commits at lists.llvm.org
Tue Jun 7 17:41:16 PDT 2022


  Branch: refs/heads/release/14.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 483db58f3eb5c625615e212283800cea2cb04b3e
      https://github.com/llvm/llvm-project/commit/483db58f3eb5c625615e212283800cea2cb04b3e
  Author: Brooks Davis <brooks at FreeBSD.org>
  Date:   2022-06-07 (Tue, 07 Jun 2022)

  Changed paths:
    M compiler-rt/lib/builtins/atomic.c

  Log Message:
  -----------
  compiler-rt: Allow build without __c11_atomic_fetch_nand

Don't build atomic fetch nand libcall functions when the required
compiler builtin isn't available. Without this compiler-rt can't be
built with LLVM 13 or earlier.

Not building the libcall functions isn't optimal, but aligns with the
usecase in FreeBSD where compiler-rt from LLVM 14 is built with an LLVM
13 clang and no LLVM 14 clang is built.

Reviewed By:	efriedma

Differential Revision:	https://reviews.llvm.org/D126710

(cherry picked from commit 18efa420da5fa065d88ba451cdcdbd950a2090c1)


  Commit: 4d039a7a71899038b3bc6ed6fe5a8a48d915caa0
      https://github.com/llvm/llvm-project/commit/4d039a7a71899038b3bc6ed6fe5a8a48d915caa0
  Author: Sergei Trofimovich <slyich at gmail.com>
  Date:   2022-06-07 (Tue, 07 Jun 2022)

  Changed paths:
    M llvm/include/llvm/Support/Signals.h

  Log Message:
  -----------
  [Support] Add missing <cstdint> header to Signals.h

Without the change llvm build fails on this week's gcc-13 snapshot as:

    [  0%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o
    In file included from llvm/lib/Support/Signals.cpp:14:
    llvm/include/llvm/Support/Signals.h:119:8: error: variable or field 'CleanupOnSignal' declared void
      119 |   void CleanupOnSignal(uintptr_t Context);
          |        ^~~~~~~~~~~~~~~

(cherry picked from commit ff1681ddb303223973653f7f5f3f3435b48a1983)


  Commit: a517f3439671f18327ada0f5d8ca328934cc337e
      https://github.com/llvm/llvm-project/commit/a517f3439671f18327ada0f5d8ca328934cc337e
  Author: Sergei Trofimovich <slyich at gmail.com>
  Date:   2022-06-07 (Tue, 07 Jun 2022)

  Changed paths:
    M llvm/include/llvm/Support/Base64.h

  Log Message:
  -----------
  [Support] Add missing <cstdint> header to Base64.h

Without the change llvm build fails on this week's gcc-13 snapshot as:

    [ 91%] Building CXX object unittests/Support/CMakeFiles/SupportTests.dir/Base64Test.cpp.o
    In file included from llvm/unittests/Support/Base64Test.cpp:14:
    llvm/include/llvm/Support/Base64.h: In function 'std::string llvm::encodeBase64(const InputBytes&)':
    llvm/include/llvm/Support/Base64.h:29:5: error: 'uint32_t' was not declared in this scope
       29 |     uint32_t x = ((unsigned char)Bytes[i] << 16) |
          |     ^~~~~~~~

(cherry picked from commit 5e9be93566f39ee6cecd579401e453eccfbe81e5)


Compare: https://github.com/llvm/llvm-project/compare/576e5b39ae4d...a517f3439671


More information about the All-commits mailing list