Changes

Summary

  1. [ELF] Replace noneRel = R_*_NONE with static constexpr. NFC (details)
  2. [ELF] Remove unneeded binding parameter from addOptionalRegular. NFC (details)
  3. [ORC] Fix uninitialized variable. (details)
  4. [ORC] Wait for handleDisconnect to complete in SimpleRemoteEPC::disconnect. (details)
  5. MLIR can't support -Bsymbolic link option, fail at CMake time with a helpful message instead of broken runtime (details)
  6. [ADT]  Add trailing comma on TYPED_TEST_SUITE (details)
Commit d23fd8ae8906866244482ee623288a657b6c5e9a by i
[ELF] Replace noneRel = R_*_NONE with static constexpr. NFC

All architectures define R_*_NONE to 0.
The file was modifiedlld/ELF/Arch/Mips.cpp
The file was modifiedlld/ELF/Arch/SPARCV9.cpp
The file was modifiedlld/ELF/Arch/AMDGPU.cpp
The file was modifiedlld/ELF/Arch/X86.cpp
The file was modifiedlld/ELF/Target.h
The file was modifiedlld/ELF/Arch/X86_64.cpp
The file was modifiedlld/ELF/Arch/PPC64.cpp
The file was modifiedlld/ELF/Arch/AVR.cpp
The file was modifiedlld/ELF/Arch/AArch64.cpp
The file was modifiedlld/ELF/Arch/PPC.cpp
The file was modifiedlld/ELF/Arch/Hexagon.cpp
The file was modifiedlld/ELF/Arch/ARM.cpp
The file was modifiedlld/ELF/Arch/RISCV.cpp
Commit db6a00daa081a1664c4b714e28042d3803b7e1f4 by i
[ELF] Remove unneeded binding parameter from addOptionalRegular. NFC

__rela_iplt_start uses spurious STB_WEAK, but it doesn't matter because STV_HIDDEN overrides the binding.
The file was modifiedlld/ELF/Writer.cpp
Commit 0371049277912afc201da721fa659ecef7ab7fba by Lang Hames
[ORC] Fix uninitialized variable.

Spotted by Dave Blaikie. Thanks Dave!
The file was modifiedllvm/lib/ExecutionEngine/Orc/Shared/SimpleRemoteEPCUtils.cpp
Commit 320832cc9b7e7fea5fc8afbed75c34c4a43287ba by Lang Hames
[ORC] Wait for handleDisconnect to complete in SimpleRemoteEPC::disconnect.

Disconnect should block until handleDisconnect completes, otherwise we might
destroy the SimpleRemoteEPC instance while it's still in use.

Thanks to Dave Blaikie for helping me track this down.
The file was modifiedllvm/lib/ExecutionEngine/Orc/SimpleRemoteEPC.cpp
The file was modifiedllvm/include/llvm/ExecutionEngine/Orc/SimpleRemoteEPC.h
Commit c3aed0d395950c23d66c3ec0af4b1a2ec2534795 by joker.eph
MLIR can't support -Bsymbolic link option, fail at CMake time with a helpful message instead of broken runtime

Differential Revision: https://reviews.llvm.org/D110483
The file was modifiedmlir/CMakeLists.txt
Commit e21b0ba8c9378bca01d2311be4e1b6ccd3397bc4 by arames
[ADT]  Add trailing comma on TYPED_TEST_SUITE

This avoids a -pedantic warning:
warning: ISO C++11 requires at least one argument for the "..." in a variadic macro

See also https://github.com/google/googletest/issues/2271

Reviewed By: arames, bkramer

Differential Revision: https://reviews.llvm.org/D110283
The file was modifiedllvm/unittests/ADT/SequenceTest.cpp