SuccessChanges

Summary

  1. Add read-only data assembly writing for aix (details)
  2. [clangd] Fix SelectionTree behavior on constructor init-lists. (details)
  3. [clangd] Don't consider class template params part of constructor name. (details)
  4. [scudo][standalone] Enabled SCUDO_DEBUG for tests + fixes (details)
  5. [clangd] Fix typo in symbol kind conversion (details)
Commit 3dfa975fb36fda1d8cff700d4251db330c83bfa3 by diggerlin
Add read-only data assembly writing for aix
SUMMARY: The patch will emit read-only variable assembly code for aix.
Reviewers: daltenty,Xiangling_Liao Subscribers: rupprecht,
seiyai,hiraditya
Differential Revision: https://reviews.llvm.org/D70182
The file was modifiedllvm/lib/MC/MCSectionXCOFF.cpp
The file was modifiedllvm/lib/BinaryFormat/XCOFF.cpp
The file was modifiedllvm/lib/MC/MCObjectFileInfo.cpp
The file was modifiedllvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
The file was modifiedllvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp
Commit ad9fd320091d44d4b8782c28b72a7be21a2bd68d by sam.mccall
[clangd] Fix SelectionTree behavior on constructor init-lists.
Summary: For the constructor Foo() : classmember(arg) {}
The AST looks like:
- CXXCtorInitializer  classmember(arg)
  - CXXConstructExpr  classmember(arg)
    - DeclRefExpr:                arg
We want the 'classmember' to be associated with the CXXCtorInitializer,
not the CXXConstructExpr. (CXXConstructExpr is known to have bad
ranges). So just early-claim it.
Thanks @hokein for tracking down/reducing the bug.
Reviewers: hokein
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet,
usaxena95, cfe-commits, hokein
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70312
The file was modifiedclang-tools-extra/clangd/unittests/SelectionTests.cpp
The file was modifiedclang-tools-extra/clangd/Selection.cpp
Commit 713c30b389602eda5c70b696e8c640487cc8b2cb by sam.mccall
[clangd] Don't consider class template params part of constructor name.
Summary: This is shorter and usually the extra info is noise. There are
cases where the params become type-parameter-0-0 that are hard to fix.
This affects a few features:
- 'name' field in structured hover API (not exposed yet)
- 'name' field in locateSymbolAt (not exposed in LSP)
- 'document/symbol' - the symbol is hierarchically nested in the class
  template, or written as foo<t>::foo when defined out-of-line.
Added a test case for hover from
https://github.com/clangd/clangd/issues/76. This patch fixes one field,
but no fewer than four others are wrong! I'll fix them...
Reviewers: hokein
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet,
usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70308
The file was modifiedclang-tools-extra/clangd/XRefs.cpp
The file was modifiedclang-tools-extra/clangd/unittests/XRefsTests.cpp
The file was modifiedclang-tools-extra/clangd/AST.cpp
Commit f018246c20481d222af4bab1868e8903c35c73d2 by kostyak
[scudo][standalone] Enabled SCUDO_DEBUG for tests + fixes
Summary:
`SCUDO_DEBUG` was not enabled for unit tests, meaning the `DCHECK`s were
never tripped. While turning this on, I discovered that a few of those
not-exercised checks were actually wrong. This CL addresses those
incorrect checks.
Not that to work in tests `CHECK_IMPL` has to explicitely use the
`scudo` namespace. Also changes a C cast to a C++ cast.
Reviewers: hctim, pcc, cferris, eugenis, vitalybuka
Subscribers: mgorny, #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D70276
The file was modifiedcompiler-rt/lib/scudo/standalone/size_class_map.h
The file was modifiedcompiler-rt/lib/scudo/standalone/tests/CMakeLists.txt
The file was modifiedcompiler-rt/lib/scudo/standalone/secondary.h
The file was modifiedcompiler-rt/lib/scudo/standalone/internal_defs.h
The file was modifiedcompiler-rt/lib/scudo/standalone/vector.h
The file was modifiedcompiler-rt/lib/scudo/standalone/combined.h
Commit c9081968ead183ee1df824f7b96fcafcfcbe57cd by sam.mccall
[clangd] Fix typo in symbol kind conversion
Reviewers: hokein
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet,
usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D70309
The file was modifiedclang-tools-extra/clangd/Protocol.cpp
The file was modifiedclang-tools-extra/clangd/unittests/XRefsTests.cpp
The file was modifiedclang-tools-extra/clangd/unittests/FindSymbolsTests.cpp