Changes

Summary

  1. Fix riscv/lld test interaction (details)
  2. [llvm-readobj] - Report a warning when an unexpected DT_SYMENT tag value is met. (details)
  3. [CGP] Add uaddo test with math used, SPARC/AArch64 variants. (details)
  4. [lldb][test] Remove expected failure decorator from test_copy_from_dummy_target (TestScriptedResolver) (details)
  5. [lldb] Fix another instance where we pass a nullptr as TypeSourceInfo to NonTypeTemplateParmDecl::Create (details)
  6. llvm: Use quotes around MSVC_DIA_SDK_DIR CMake variable (details)
Commit cc7a197f9f1039e36625249199aa652f84506802 by sguelton
Fix riscv/lld test interaction

Fix for dd230142d8a00f5f30c3930a2407000e845dcfbf, in case ld.lld is not
available.
The file was modifiedclang/test/Driver/riscv32-toolchain.c (diff)
Commit b87a0f7416f1a657d808690295b44669849e7bfc by grimar
[llvm-readobj] - Report a warning when an unexpected DT_SYMENT tag value is met.

There was a short discussion about this:
https://reviews.llvm.org/D73484#inline-676942

To summarize:
It is a bit unclear to me why the `DT_SYMENT` tag exist.
LLD has the code that does:
"addInt(DT_SYMENT, sizeof(Elf_Sym));" and I guess other linkers has the same logic.
It is unclear why it can be possible to have other values rather than values of
a size of platform symbol. Seems it is not possible, and atm for me it looks that
this tag should not be used. This patch starts reporting the warning when the
value it contains differs from a symbol size for a 32/64 bit platform for safety.
It keeps the rest of the logic we have unchanged. Before this patch we did not handle
the tag at all.

Differential review: https://reviews.llvm.org/D74479
The file was modifiedllvm/test/tools/llvm-readobj/ELF/dyn-symbols.test (diff)
The file was modifiedllvm/tools/llvm-readobj/ELFDumper.cpp (diff)
Commit 106ae108c18fb13e298e39dd010edcc87a10e48d by flo
[CGP] Add uaddo test with math used, SPARC/AArch64 variants.
The file was modifiedllvm/test/Transforms/CodeGenPrepare/X86/overflow-intrinsics.ll (diff)
The file was addedllvm/test/Transforms/CodeGenPrepare/SPARC/overflow-intrinsics.ll
The file was addedllvm/test/Transforms/CodeGenPrepare/SPARC/lit.local.cfg
The file was addedllvm/test/Transforms/CodeGenPrepare/AArch64/overflow-intrinsics.ll
Commit db23825970b021d0b5302d43aa5355c295c133d9 by Tatyana Krasnukha
[lldb][test] Remove expected failure decorator from test_copy_from_dummy_target (TestScriptedResolver)

This test case doesn't check that breakpoint's locations are resolved, and it passes on Windows too.
The file was modifiedlldb/test/API/functionalities/breakpoint/scripted_bkpt/TestScriptedResolver.py (diff)
Commit 40b2c7f29b44e3c3e171a04593ac650aaf7c6ae6 by Raphael Isemann
[lldb] Fix another instance where we pass a nullptr as TypeSourceInfo to NonTypeTemplateParmDecl::Create

Summary:
Follow up to an issue pointed out in the review of D73808. We shouldn't just pass in a nullptr TypeSourceInfo
in case Clang decided to access it.

Reviewers: shafik, vsk

Reviewed By: shafik, vsk

Subscribers: kristof.beyls, JDevlieghere, lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D73946
The file was modifiedlldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp (diff)
Commit d6fe253653b7e760f94ca21d0a7ebbfeeee28777 by martin
llvm: Use quotes around MSVC_DIA_SDK_DIR CMake variable

MSVC_DIA_SDK_DIR variable will point to a path which contains spaces,
and without quotes it will fail to configure the project.
The file was modifiedllvm/cmake/config-ix.cmake (diff)