[all-commits] [llvm/llvm-project] b54184: [clang] [Windows] Mark PIC as implicitly enabled f...

[Martin Storsjö via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b541845ea082d1dfca53074b4353e1ee0c22dd76
      https://github.com/llvm/llvm-project/commit/b541845ea082d1dfca53074b4353e1ee0c22dd76
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/MSVC.cpp
    M clang/lib/Driver/ToolChains/MinGW.cpp
    M clang/test/Driver/pic.c

  Log Message:
  -----------
  [clang] [Windows] Mark PIC as implicitly enabled for aarch64, just like for x86_64

This doesn't practically affect the code generation.

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


  Commit: a03e17d4d9419811da16bc2e893fd0cf9b8aa2a7
      https://github.com/llvm/llvm-project/commit/a03e17d4d9419811da16bc2e893fd0cf9b8aa2a7
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M libcxx/test/std/numerics/bit/bit.cast/bit_cast.pass.cpp

  Log Message:
  -----------
  [libcxx] [test] Generalize the conditions for testing bitcasts between long double, double and int128

MSVC targets also have a 64 bit long double, as do MinGW targets on ARM.
This hasn't been noticed in CI because the MSVC configurations there run
with _LIBCPP_HAS_NO_INT128 defined.

This avoids assuming that either __int128_t or double is equal in size to
long double. i386 MinGW targets have sizeof(long double) == 10, which
doesn't match any of the tested types.

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


  Commit: d9b9a7f4287019ad7fb5ae35523e81dee36c1b40
      https://github.com/llvm/llvm-project/commit/d9b9a7f4287019ad7fb5ae35523e81dee36c1b40
  Author: Jeremy Drake <github at jdrake.com>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M clang/lib/Tooling/JSONCompilationDatabase.cpp

  Log Message:
  -----------
  [clang][Tooling] Use Windows command lines on all Windows, except Cygwin

Previously it only used Windows command lines for MSVC triples, but this
was causing issues for windows-gnu.  In fact, everything 'native' Windows
(ie, not Cygwin) should use Windows command line parsing.

Reviewed By: mstorsjo

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


  Commit: 2a4b1539e991f289ea62f74a03f3fc6aa49c9c1f
      https://github.com/llvm/llvm-project/commit/2a4b1539e991f289ea62f74a03f3fc6aa49c9c1f
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

  Changed paths:
    M llvm/include/llvm/Support/Path.h
    M llvm/lib/Support/Path.cpp

  Log Message:
  -----------
  [Support] [Path] Use std::replace instead of an explicit comparison loop. NFC.

After 8fc7a907b93a8e9eef96e872f8f926db3ebfe9b6, this loop does
the same as a plain `std::replace`.

Also clarify the comment about what this function does.

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


  Commit: 6fbc812883ab949d2ce8ee8768f986f2cb39ac35
      https://github.com/llvm/llvm-project/commit/6fbc812883ab949d2ce8ee8768f986f2cb39ac35
  Author: Martin Storsjö <martin at martin.st>
  Date:   2021-10-13 (Wed, 13 Oct 2021)

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

  Log Message:
  -----------
  [Support] [Path] Move function declarations to the right doxygen group in the header. NFC.

They were in the doxygen group Observers, while they are about
mutating paths.

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


Compare: https://github.com/llvm/llvm-project/compare/bde89ac7f1da...6fbc812883ab