SuccessChanges

Summary

  1. [lldb] Remove RTTI in ClangExternalASTSourceCommon based on a global map (details)
  2. [Kaleidoscope][BuildingAJIT] Add a missing std::move. (details)
  3. [MC] Delete unused MCAsmInfoELF::UsesNonexecutableStackSection after (details)
  4. [MC] Ignore VK_WEAKREF in MCValue::getAccessVariant (details)
  5. [MC] Assume CommentStream is non-null in MCDisassembler::tryAdding* (details)
  6. [ORC] Make ObjectLinkingLayer own its jitlink::MemoryManager. (details)
  7. [PowerPC] Fix %llvm.ppc.altivec.vc* lowering (details)
Commit 8280896bd1b055a192d9e7d482b0ffa14ee88e3a by Raphael Isemann
[lldb] Remove RTTI in ClangExternalASTSourceCommon based on a global map
of known instances
Summary: Currently we do our RTTI check for ClangExternalASTSourceCommon
by using this global map of ClangExternalASTSourceCommon where every
instance is registering and deregistering itself on
creation/destruction. Then we can do the RTTI check by looking up in
this map from ClangASTContext.
This patch removes this whole thing and just adds LLVM-style RTTI
support to ClangExternalASTSourceCommon which is possible with D71397.
Reviewers: labath, aprantl
Reviewed By: labath
Subscribers: JDevlieghere, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D71398
The file was modifiedlldb/source/Symbol/ClangExternalASTSourceCommon.cpp (diff)
The file was modifiedlldb/source/Symbol/ClangASTContext.cpp (diff)
The file was modifiedlldb/include/lldb/Symbol/ClangExternalASTSourceCommon.h (diff)
Commit 064e7d1df17dcd7f82215c258e2793aa1f8ed760 by Lang Hames
[Kaleidoscope][BuildingAJIT] Add a missing std::move.
This should fix the build failure described in
https://github.com/llvm/llvm-project/issues/63.
The file was modifiedllvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/KaleidoscopeJIT.h (diff)
Commit fdb408f348f91f8eb4ecc32527aebfb6286e55cb by maskray
[MC] Delete unused MCAsmInfoELF::UsesNonexecutableStackSection after
EM_WEBASSEMBLY was removed in D48744
This removes remnant of D15969 which hasn't been removed by D48744.
The file was removedllvm/test/CodeGen/WebAssembly/non-executable-stack.ll
The file was modifiedllvm/include/llvm/MC/MCAsmInfoELF.h (diff)
The file was modifiedllvm/lib/MC/MCAsmInfoELF.cpp (diff)
Commit 2b0256e49bbe5c0dc9c8f4800b1e2f131026cb45 by maskray
[MC] Ignore VK_WEAKREF in MCValue::getAccessVariant
MCSymbolRefExpr::getVariantKindForName does not return VK_WEAKREF, so
this code path is not exercised. Moreoever, .weakref is probably a
feature that nobody uses.
The file was modifiedllvm/lib/MC/MCValue.cpp (diff)
Commit 1ea5ce6335c7c0e040f70de16f1080546d242b69 by maskray
[MC] Assume CommentStream is non-null in MCDisassembler::tryAdding*
AArch64/ARM/X86 call the two functions. CommentStream is always
initialized.
The file was modifiedllvm/lib/MC/MCDisassembler/MCDisassembler.cpp (diff)
Commit c0143f37dab38a2da27c9bb6f44f3a54ca9cf86f by Lang Hames
[ORC] Make ObjectLinkingLayer own its jitlink::MemoryManager.
This relieves ObjectLinkingLayer clients of the responsibility of
holding the memory manager. This makes it easier to select between
RTDyldObjectLinkingLayer
(which already owned its memory manager factory) and ObjectLinkingLayer
at runtime as clients aren't required to hold a jitlink::MemoryManager
field just in case ObjectLinkingLayer is selected.
The file was modifiedllvm/lib/ExecutionEngine/Orc/ObjectLinkingLayer.cpp (diff)
The file was modifiedllvm/tools/llvm-jitlink/llvm-jitlink.h (diff)
The file was modifiedllvm/tools/llvm-jitlink/llvm-jitlink.cpp (diff)
The file was modifiedllvm/include/llvm/ExecutionEngine/Orc/ObjectLinkingLayer.h (diff)
Commit 7e0fd77645cf6eae1994b255a307b016180c8a83 by tclin914
[PowerPC] Fix %llvm.ppc.altivec.vc* lowering
Summary: r372285 changed LLVM to use a `TargetConstant` for parameters
of intrinsics that are required to be immediates.
Since that commit, use of `%llvm.ppc.altivec.vc{fsx,fux,tsxs,tuxs}`
intrinsics has not worked, and resulted in a `LLVM ERROR: Cannot select:
intrinsic %llvm.ppc.altivec.vc*` error. The intrinsics' TableGen
definitions matched on `imm` instead of `timm`.
This commit updates those definitions to use `timm`.
Fixes: https://llvm.org/PR44239
Reviewers: hfinkel, nemanjai, #powerpc, Jim
Reviewed By: Jim
Subscribers: qiucf, wuzish, Jim, hiraditya, kbarton, jsji, shchenz,
llvm-commits
Tags: #llvm
Patched by vddvss (Colin Samples).
Differential Revision: https://reviews.llvm.org/D71138
The file was addedllvm/test/CodeGen/PowerPC/pr44239.ll
The file was modifiedllvm/lib/Target/PowerPC/PPCInstrAltivec.td (diff)