SuccessChanges

Summary

  1. [clangd] Log use of heuristic go-to-def. NFC (details)
  2. [OpenMP] Fix a race in task queue reallocation (details)
  3. [LV] Clamp MaxVF to power of 2. (details)
  4. [AST] default implementation is possible for non-member functions in C++20. (details)
  5. [clangd] Enable cross-file-rename by default. (details)
Commit b752a2743ab0d24d8da5d97c07fbdb996df78b1f by sam.mccall
[clangd] Log use of heuristic go-to-def. NFC

Generally:
- found results using this method -> log
- no results using this method -> vlog
- method wasn't applied because ineligible -> no log
The file was modifiedclang-tools-extra/clangd/XRefs.cpp
Commit 3895148d7cd8ff76220f8f8209ec06369a8e816f by protze
[OpenMP] Fix a race in task queue reallocation

__kmp_realloc_task_deque implicitly assumes, that the task queue is full
(ntasks == size), therefore tail = size in line 319.
An assertion is added to document this assumption.

The first check for a full queue is before the locking and might not hold
when the lock is taken. So, we need to check again for this condition when
we have the lock.

Reviewed By: AndreyChurbanov

Differential Revision: https://reviews.llvm.org/D80480
The file was modifiedopenmp/runtime/src/kmp_tasking.cpp
Commit 840450549c9199150cbdee29acef756c19660ca1 by ayal.zaks
[LV] Clamp MaxVF to power of 2.

If a loop has a constant trip count known to be a multiple of MaxVF (times user
UF), LV infers that no tail will be generated for any chosen VF. This relies on
the chosen VF's being powers of 2 bound by MaxVF, and assumes MaxVF is a power
of 2. Make sure the latter holds, in particular when MaxVF is set by a memory
dependence distance which may not be a power of 2.

Differential Revision: https://reviews.llvm.org/D80491
The file was addedllvm/test/Transforms/LoopVectorize/memdep-fold-tail.ll
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorize.cpp
Commit 447ea9b4f5f562c8fab7d11ecbb10ecd33155d5b by hokein.wu
[AST] default implementation is possible for non-member functions in C++20.

Summary:
Make RAV not visit the default function decl by default.
Also update some stale comments on FunctionDecl::isDefault.

Fixes https://github.com/clangd/clangd/issues/383

Reviewers: sammccall, rsmith

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80288
The file was modifiedclang/include/clang/AST/RecursiveASTVisitor.h
The file was modifiedclang/include/clang/AST/Decl.h
The file was modifiedclang/unittests/Tooling/RecursiveASTVisitorTests/CXXMethodDecl.cpp
Commit 72c5ea1d73bb89af6f82c14ddb0b7f4c2510bab7 by hokein.wu
[clangd] Enable cross-file-rename by default.

Summary:
The cross-file rename feature is stable enough to enable it (has been
rolled out internally for a few weeks).

Reviewers: sammccall

Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80507
The file was modifiedclang-tools-extra/clangd/tool/ClangdMain.cpp