FailedChanges

Summary

  1. Solaris buildbot updates (details)
Commit cc31e521f3cad60bf446eded4bc7a55cf28d5998 by ro
Solaris buildbot updates

Once D84029 <https://reviews.llvm.org/D84029> lands, it's no longer
necessary to set `CLANG_DEFAULT_LINKER`.  On the contrary, it causes test
failures since D84029 <https://reviews.llvm.org/D84029>.

The patch also changes the Solaris/amd64 `LLVM_HOST_TRIPLE` to match the
bot name and Solaris conventions and fixes a pasto in the Solaris/sparcv9
triple.

Differential Revision: https://reviews.llvm.org/D85462
The file was modifiedbuildbot/osuosl/master/config/builders.py (diff)

Summary

  1. [ORC][NFC] Fix typo in comment (details)
  2. [gn build] (manually) port d650cbc349ccc4 (details)
  3. [gn build] (manually) port d650cbc349ccc4 better (details)
  4. [SVE] Lower fixed length vector integer UMIN/UMAX (details)
  5. [libc] Add strtok_r implementation. (details)
  6. [tsan] Respect no_huge_pages_for_shadow. (details)
  7. [SystemZ][z/OS] enable trigraphs by default on z/OS (details)
  8. Add "status" to the list of absl libraries. (details)
  9. sanitizer_common: Introduce internal_madvise and start using it. (details)
  10. [InstSimplify] Add tests for assume with min/max intrinsic (NFC) (details)
  11. [SLC] Optimize strncpy(a, a, C) to memcpy(a, a000, C) (details)
  12. Re-apply "[llvm-jitlink] Don't demote unreferenced definitions in -harness mode" (details)
  13. [Tests] Update strncpy tests (details)
  14. [clang][Driver] Default to /usr/bin/ld on Solaris (details)
  15. [WebAssembly] Allow inlining functions with different features (details)
Commit 5bcd32b7449482d6079d968c08d07a3890c86912 by Stefan Gränitz
[ORC][NFC] Fix typo in comment
The file was modifiedllvm/lib/ExecutionEngine/Orc/LLJIT.cpp
Commit 5092039644eac33bdd9e494045a9f99654f792b3 by thakis
[gn build] (manually) port d650cbc349ccc4
The file was modifiedllvm/utils/gn/secondary/llvm/tools/llvm-elfabi/BUILD.gn
The file was addedllvm/utils/gn/secondary/llvm/lib/InterfaceStub/BUILD.gn
The file was addedllvm/utils/gn/secondary/llvm/unittests/InterfaceStub/BUILD.gn
The file was modifiedllvm/utils/gn/secondary/llvm/lib/TextAPI/BUILD.gn
The file was modifiedllvm/utils/gn/secondary/llvm/unittests/TextAPI/BUILD.gn
The file was modifiedllvm/utils/gn/secondary/llvm/unittests/BUILD.gn
Commit 661d83aa386fd2c2df769a2509f7c1bd5ba99a7b by thakis
[gn build] (manually) port d650cbc349ccc4 better
The file was modifiedllvm/utils/gn/secondary/llvm/lib/InterfaceStub/BUILD.gn
Commit 21810b0e14287a7b885a7822c6e19609e3b902c8 by mcinally
[SVE] Lower fixed length vector integer UMIN/UMAX

Differential Revision: https://reviews.llvm.org/D85926
The file was modifiedllvm/test/CodeGen/AArch64/sve-fixed-length-int-minmax.ll
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Commit bc45bab7eb6d6138442ba0dfab11faa734566ebf by parallels
[libc] Add strtok_r implementation.

Reviewed By: sivachandra

Differential Revision: https://reviews.llvm.org/D85779
The file was modifiedlibc/config/linux/aarch64/entrypoints.txt
The file was modifiedlibc/src/string/strtok.cpp
The file was modifiedlibc/config/linux/x86_64/entrypoints.txt
The file was modifiedlibc/config/linux/api.td
The file was modifiedlibc/src/string/CMakeLists.txt
The file was addedlibc/src/string/strtok_r.cpp
The file was modifiedlibc/spec/posix.td
The file was modifiedlibc/src/string/string_utils.h
The file was modifiedlibc/test/src/string/CMakeLists.txt
The file was addedlibc/test/src/string/strtok_r_test.cpp
The file was addedlibc/src/string/strtok_r.h
Commit 96855125e77044b1a5d3c7f0ae90ea3a5cb035c0 by ckennelly
[tsan] Respect no_huge_pages_for_shadow.

Disable huge pages in the TSan shadow regions when no_huge_pages_for_shadow == true (default).

Reviewed By: eugenis

Differential Revision: https://reviews.llvm.org/D85841
The file was modifiedcompiler-rt/lib/tsan/rtl/tsan_rtl.cpp
The file was modifiedcompiler-rt/lib/tsan/rtl/tsan_platform_posix.cpp
The file was modifiedcompiler-rt/lib/tsan/rtl/tsan_sync.cpp
Commit dedaf78fa71433d3c9da2e3d2f3dad3e9cd3bdd2 by hubert.reinterpretcast
[SystemZ][z/OS] enable trigraphs by default on z/OS

This patch enables trigraphs on z/OS.

Reviewed By: hubert.reinterpretcast, fanbo-meng

Differential Revision: https://reviews.llvm.org/D85722
The file was modifiedclang/test/Frontend/trigraphs.cpp
The file was modifiedclang/test/Lexer/cxx1z-trigraphs.cpp
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
Commit 114c9fa0e46f7bf1d05d92da70da116b19f16911 by vyng
Add "status" to the list of absl libraries.

The Abseil-NoInternalDependenciesCheck currently mistakenly triggers on any usage of internal helpers even if it is within absl/status.

Differential Revision: https://reviews.llvm.org/D85843
The file was modifiedclang-tools-extra/clang-tidy/abseil/AbseilMatcher.h
Commit 9f8c4039f202c4f8e8b820bce6f05039d53ea005 by peter
sanitizer_common: Introduce internal_madvise and start using it.

A recent change to sanitizer_common caused us to issue the syscall
madvise(MADV_HUGEPAGE) during HWASAN initialization. This may lead to a
problem if madvise is instrumented (e.g. because libc is instrumented
or the user intercepted it). For example, on Android the syscall may
fail if the kernel does not support transparent hugepages, which leads
to an attempt to set errno in a HWASAN instrumented function. Avoid
this problem by introducing a syscall wrapper and using it to issue
this syscall.

Tested only on Linux; includes untested updates for the other
platforms.

Differential Revision: https://reviews.llvm.org/D85870
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_mac.cpp
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_netbsd.cpp
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_solaris.cpp
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_posix.h
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_posix_libcdep.cpp
The file was modifiedcompiler-rt/lib/sanitizer_common/sanitizer_openbsd.cpp
Commit 524f591a395cbb65a67c85b4df111c5f4e9d753e by nikita.ppv
[InstSimplify] Add tests for assume with min/max intrinsic (NFC)

If we assume one of the operands is smaller/greater, then min/max
may be simplified.
The file was modifiedllvm/test/Transforms/InstSimplify/maxmin_intrinsics.ll
Commit 5ef2287d36f93bac3b7b8c49c1b371d25d305e85 by Dávid Bolvanský
[SLC] Optimize strncpy(a, a, C) to memcpy(a, a000, C)
Solves PR47154
The file was modifiedllvm/lib/Transforms/Utils/SimplifyLibCalls.cpp
The file was addedllvm/test/Transforms/InstCombine/strncpy-3.ll
Commit f136e5ecfb335c61a23f09187150f363919df11a by Lang Hames
Re-apply "[llvm-jitlink] Don't demote unreferenced definitions in -harness mode"

This reapplies commit e137b550587a85b0d9c9c539edc79de0122b6946 with
fixes for the broken test case: Non-global symbols should only be
skipped after checking that they're not referenced by the harness.
The file was modifiedllvm/tools/llvm-jitlink/llvm-jitlink.cpp
Commit 18910c4cb5d4e59ef25f4b8b48fcbd3146833645 by Dávid Bolvanský
[Tests] Update strncpy tests
The file was modifiedllvm/test/Transforms/InstCombine/strncpy-1.ll
Commit f59bec7acb8228fc215fca3ee1e524c38083c50b by ro
[clang][Driver] Default to /usr/bin/ld on Solaris

`clang` currently requires the native linker on Solaris:

  - It passes `-C` to `ld` which GNU `ld` doesn't understand.

  - To use `gld`, one needs to pass the correct `-m EMU` option to select
    the right emulation.  Solaris `ld` cannot handle that option.

So far I've worked around this by passing `-DCLANG_DEFAULT_LINKER=/usr/bin/ld`
to `cmake`.  However, if someone forgets this, it depends on the user's
`PATH` whether or not `clang` finds the correct linker, which doesn't make
for a good user experience.

While it would be nice to detect the linker flavor at runtime, this is more
involved.  Instead, this patch defaults to `/usr/bin/ld` on Solaris.  This
doesn't work on its own, however: a link fails with

  clang-12: error: unable to execute command: Executable "x86_64-pc-solaris2.11-/usr/bin/ld" doesn't exist!

I avoid this by leaving absolute paths alone in `ToolChain::GetLinkerPath`.

Tested on `amd64-pc-solaris2.11`, `sparcv9-sun-solaris2.11`, and
`x86_64-pc-linux-gnu`.

Differential Revision: https://reviews.llvm.org/D84029
The file was modifiedclang/lib/Driver/ToolChains/Solaris.h
The file was addedclang/test/Driver/solaris-ld-sld.c
The file was modifiedclang/lib/Driver/ToolChain.cpp
The file was modifiedllvm/utils/lit/lit/llvm/config.py
Commit d53d952810e7afaad66a8c527fa073e8f68f6222 by tlively
[WebAssembly] Allow inlining functions with different features

Allow inlining only when the Callee has a subset of the Caller's
features. In principle, we should be able to inline regardless of any
features because WebAssembly supports features at module granularity,
not function granularity, but without this restriction it would be
possible for a module to "forget" about features if all the functions
that used them were inlined.

Requested in PR46812.

Differential Revision: https://reviews.llvm.org/D85494
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.cpp
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.h
The file was addedllvm/test/Transforms/Inline/WebAssembly/inline-target-features.ll

Summary

  1. Solaris buildbot updates (details)
Commit cc31e521f3cad60bf446eded4bc7a55cf28d5998 by ro
Solaris buildbot updates

Once D84029 <https://reviews.llvm.org/D84029> lands, it's no longer
necessary to set `CLANG_DEFAULT_LINKER`.  On the contrary, it causes test
failures since D84029 <https://reviews.llvm.org/D84029>.

The patch also changes the Solaris/amd64 `LLVM_HOST_TRIPLE` to match the
bot name and Solaris conventions and fixes a pasto in the Solaris/sparcv9
triple.

Differential Revision: https://reviews.llvm.org/D85462
The file was modifiedbuildbot/osuosl/master/config/builders.py