SuccessChanges

Summary

  1. Revert "[CMake] Use find_dependency in LLVMConfig.cmake" (details)
  2. Reapply: [clang-cl] Always interpret the LIB env var as separated with semicolons (details)
  3. [AArch64] Fix return type of Neon scalar comparison intrinsics (details)
Commit 6fa38de071f0993be81ea090c4a307b1d146c179 by phosek
Revert "[CMake] Use find_dependency in LLVMConfig.cmake"

This reverts commit 247c4fc50720ed48db2464bbe59839eedbe16794 as it
broke the runtime build.
The file was modifiedllvm/cmake/modules/LLVMConfig.cmake.in (diff)
Commit 3fec6ddc276a595e4409f04dabdd50c84f5f2a2d by martin
Reapply: [clang-cl] Always interpret the LIB env var as separated with semicolons

When cross compiling with clang-cl, clang splits the INCLUDE env
variable around semicolons (clang/lib/Driver/ToolChains/MSVC.cpp,
MSVCToolChain::AddClangSystemIncludeArgs) and lld splits the
LIB variable similarly (lld/COFF/Driver.cpp,
LinkerDriver::addLibSearchPaths). Therefore, the consensus for
cross compilation with clang-cl and lld-link seems to be to use
semicolons, despite path lists normally being separated by colons
on unix and EnvPathSeparator being set to that.

Therefore, handle the LIB variable similarly in Clang, when
handling lib file arguments when driving linking via Clang.

This fixes commands like "clang-cl test.c -Fetest.exe kernel32.lib" in
a cross compilation setting. Normally, most users call (lld-)link
directly, but meson happens to use this command syntax for
has_function() tests.

Reapply: Change Program.h to define procid_t as ::pid_t. When included
in lldb/unittests/Host/NativeProcessProtocolTest.cpp, it is included
after an lldb namespace containing an lldb::pid_t typedef, followed
later by a "using namespace lldb;". Previously, Program.h wasn't
included in this translation unit, but now it ends up included
transitively from Process.h.

Differential Revision: https://reviews.llvm.org/D88002
The file was modifiedclang/test/Driver/cl-inputs.c (diff)
The file was modifiedllvm/include/llvm/Support/Program.h (diff)
The file was modifiedllvm/include/llvm/Support/Process.h (diff)
The file was modifiedllvm/lib/Support/Process.cpp (diff)
The file was modifiedclang/lib/Driver/Driver.cpp (diff)
Commit f93514545cd91b132fe987618488b8c1e5388fb0 by david.spickett
[AArch64] Fix return type of Neon scalar comparison intrinsics

The following should have unsigned return types
but were signed:
vceqd_s64 vceqzd_s64 vcged_s64 vcgezd_s64
vcgtd_s64 vcgtzd_s64 vcled_s64 vclezd_s64
vcltd_s64 vcltzd_s64 vtstd_s64

See https://developer.arm.com/documentation/ihi0073/latest

Reviewed By: efriedma

Differential Revision: https://reviews.llvm.org/D88009
The file was modifiedclang/include/clang/Basic/arm_neon.td (diff)
The file was modifiedclang/test/CodeGen/aarch64-neon-intrinsics.c (diff)