SuccessChanges

Changes from Git (git http://labmaster3.local/git/llvm-project.git)

Summary

  1. [clangd] Fix path edge-case condition. (details)
  2. [libc++] hash<long double>: adjust for x86-64 ILP32 (details)
  3. [IR] remove redundant code comments; NFC (details)
  4. [IR] simplify code in removePredecessor(); NFCI (details)
  5. [VPlan] Manage stored values of interleave groups using VPUser (NFC) (details)
  6. [OpenMP][OMPT][NFC] Fix flaky test (details)
  7. [OpenMP][OMPT][NFC] Fix failing test (details)
  8. Use `const` for array pointers in `StandardTypes.h` (details)
  9. [VPlan] Use VPValue and VPUser ops to print VPReplicateRecipe. (details)
  10. [VPlan] Fix -Wunused-variable after a813090072c0527eb6ed51dd2ea4f54cb6bc72a0 (details)
  11. [mlir] Add a shape function library op (details)
  12. [RISCV] Change predicate on InstAliases for GORCI/GREVI/SHFLI/UNSHFLI to HasStdExtZbp instead of HasStdExtZbbOrZbp. (details)
  13. [ELF] Check --orphan-handling=place (default value) early (details)
  14. [DL] Optimize address space zero lookup (NFC) (details)
  15. [ELF] Improve 2 SmallVector<*, N> usage (details)
  16. [clang-tidy] [clangd] Avoid multi-line diagnostic range for else-after-return diagnostic (details)
Commit d99da80841cb4d9734db4a48cd49e37b623176bc by sam.mccall
[clangd] Fix path edge-case condition.
The file was modifiedclang-tools-extra/clangd/ConfigProvider.cpp
Commit fba0b65f727134e8d05c785b04b7b574f852d49e by harald
[libc++] hash<long double>: adjust for x86-64 ILP32

x86-64 ILP32 mode (x32) uses 32-bit size_t, so share the code with ix86 to zero out padding bits, not with x86-64 LP64 mode.

Reviewed By: #libc, ldionne

Differential Revision: https://reviews.llvm.org/D91349
The file was modifiedlibcxx/include/utility
Commit 2cebad702cdff8c320c8afa748626e8cc1b3b2f3 by spatel
[IR] remove redundant code comments; NFC

As noted in D92247 (and independent of that patch):

http://llvm.org/docs/CodingStandards.html#doxygen-use-in-documentation-comments

"Don’t duplicate the documentation comment in the header file and in the
implementation file. Put the documentation comments for public APIs into
the header file."
The file was modifiedllvm/lib/IR/BasicBlock.cpp
Commit ce134da4b18c27bbeba4e32f5813b1a3b043066e by spatel
[IR] simplify code in removePredecessor(); NFCI

As suggested in D92247 (and independent of whatever we decide to do there),
this code is confusing as-is. Hopefully, this is at least mildly better.

We might be able to do better still, but we have a function called
"removePredecessor" with this behavior:
"Note that this function does not actually remove the predecessor." (!)
The file was modifiedllvm/lib/IR/BasicBlock.cpp
Commit a813090072c0527eb6ed51dd2ea4f54cb6bc72a0 by flo
[VPlan] Manage stored values of interleave groups using VPUser (NFC)

Interleave groups also depend on the values they store. Manage the
stored values as VPUser operands. This is currently a NFC, but is
required to allow VPlan transforms and to manage generated vector values
exclusively in VPTransformState.
The file was modifiedllvm/lib/Transforms/Vectorize/VPlan.h
The file was modifiedllvm/unittests/Transforms/Vectorize/VPlanTest.cpp
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorize.cpp
Commit cdf9401df84ef382467d1ca1c1c458c11fd6043a by protze
[OpenMP][OMPT][NFC] Fix flaky test

The test had a chance to finish the first task before the second task is
created. In this case, the dependences-pair event would not trigger.
The file was modifiedopenmp/runtime/test/ompt/tasks/dependences.c
Commit 723be4042a3aa38523c60b1dd96b20448053c41e by protze
[OpenMP][OMPT][NFC] Fix failing test

The test would fail for gcc, when built with debug flag.
The file was modifiedopenmp/runtime/test/ompt/tasks/serialized.c
Commit 7f521318e4f7d9e64907fad8c4bd83ddc037f8c6 by 989903+GeorgeLyon
Use `const` for array pointers in `StandardTypes.h`

This mirrors the underlying C++ api.

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D92252
The file was modifiedmlir/include/mlir-c/StandardTypes.h
The file was modifiedmlir/lib/CAPI/IR/StandardTypes.cpp
Commit 4bc9b909d715157a2d04f32c32b828c23c1d4359 by flo
[VPlan] Use VPValue and VPUser ops to print VPReplicateRecipe.
The file was modifiedllvm/lib/Transforms/Vectorize/VPlan.cpp
The file was modifiedllvm/test/Transforms/LoopVectorize/vplan-printing.ll
Commit 5408fdcd78ad4783a39c0c18d5a18a7d926514b7 by i
[VPlan] Fix -Wunused-variable after a813090072c0527eb6ed51dd2ea4f54cb6bc72a0
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorize.cpp
Commit e534cee26ae3626ced20438ea82e11291cc768e8 by jpienaar
[mlir] Add a shape function library op

Op with mapping from ops to corresponding shape functions for those op
in the library and mechanism to associate shape functions to functions.
The mapping of operand to shape function is kept separate from the shape
functions themselves as the operation is associated to the shape
function and not vice versa, and one could have a common library of
shape functions that can be used in different contexts.

Use fully qualified names and require a name for shape fn lib ops for
now and an explicit print/parse (based around the generated one & GPU
module op ones).

This commit reverts d9da4c3e73720badfcac5c0dc63c0285bb690770. Fixes
missing headers (don't know how that was working locally).

Differential Revision: https://reviews.llvm.org/D91672
The file was modifiedmlir/include/mlir/Dialect/Shape/IR/Shape.h
The file was modifiedmlir/test/lib/Dialect/CMakeLists.txt
The file was modifiedmlir/include/mlir/Dialect/Shape/IR/ShapeOps.td
The file was addedmlir/test/lib/Dialect/Shape/CMakeLists.txt
The file was addedmlir/test/lib/Dialect/Shape/TestShapeFunctions.cpp
The file was modifiedmlir/tools/mlir-opt/mlir-opt.cpp
The file was addedmlir/test/Analysis/test-shape-fn-report.mlir
The file was modifiedmlir/lib/Dialect/Shape/IR/Shape.cpp
The file was modifiedmlir/test/lib/Dialect/Test/TestOps.td
The file was modifiedmlir/tools/mlir-opt/CMakeLists.txt
Commit 84aad9b5da96ca2fe47c9db46c3a6b1bb09c070b by craig.topper
[RISCV] Change predicate on InstAliases for GORCI/GREVI/SHFLI/UNSHFLI to HasStdExtZbp instead of HasStdExtZbbOrZbp.

This matches the predicate on the instructions. Though I think
some specific encodings are valid in Zbb, but not all of them.
The file was modifiedllvm/lib/Target/RISCV/RISCVInstrInfoB.td
Commit 048b16f7fbb745635b48d31ee957bb8865597606 by i
[ELF] Check --orphan-handling=place (default value) early

The function took 1% (161MiB clang) to 1.7% (an 4.9GiB executable) time.
The file was modifiedlld/ELF/LinkerScript.cpp
Commit 891170e8636b312092486dee7c9117db7def8836 by nikita.ppv
[DL] Optimize address space zero lookup (NFC)

Information for pointer size/alignment/etc is queried a lot, but
the binary search based implementation makes this fairly slow.

Add an explicit check for address space zero and skip the search
in that case -- we need to specially handle the zero address space
anyway, as it serves as the fallback for all address spaces that
were not explicitly defined.

I initially wanted to simply replace the binary search with a
linear search, which would handle both address space zero and the
general case efficiently, but I was not sure whether there are
any degenerate targets that use more than a handful of declared
address spaces (in-tree, even AMDGPU only declares six).
The file was modifiedllvm/include/llvm/IR/DataLayout.h
The file was modifiedllvm/lib/IR/DataLayout.cpp
Commit dfcf1acf13226be0f599a7ab6b395b66dc9545d6 by i
[ELF] Improve 2 SmallVector<*, N> usage

For --gc-sections, SmallVector<InputSection *, 256> -> SmallVector<InputSection *, 0> because the code bloat (1296 bytes) is not worthwhile (the saved reallocation is negligible).
For OutputSection::compressedData, N=1 is useless (for a compressed .debug_*, the size is always larger than 1).
The file was modifiedlld/ELF/MarkLive.cpp
The file was modifiedlld/ELF/OutputSections.h
Commit f15b7869e5afbd6c24ef440b0b62593e80fbd24f by zeratul976
[clang-tidy] [clangd] Avoid multi-line diagnostic range for else-after-return diagnostic

Fixes https://bugs.llvm.org/show_bug.cgi?id=47809

Differential Revision: https://reviews.llvm.org/D92272
The file was modifiedclang-tools-extra/clangd/unittests/DiagnosticsTests.cpp
The file was modifiedclang-tools-extra/clang-tidy/readability/ElseAfterReturnCheck.cpp