SuccessChanges

Summary

  1. [monorepo_build.py] Add "sudo" and "err_okay" args to the run_ws wrapper too (details)
  2. [monorepo_build.py] Add sccache support (details)
Commit 4aa996ae0a8a9e4c1e261932c67ed327b22ef034 by Azharuddin Mohammed
[monorepo_build.py] Add "sudo" and "err_okay" args to the run_ws wrapper too

run_cmd (for which run_ws is a wrapper) already accepts these arguments.
The file was modifiedzorg/jenkins/monorepo_build.py (diff)
Commit e4ca265465a5e1d8502fe45b4b57c297e7412ac9 by Azharuddin Mohammed
[monorepo_build.py] Add sccache support
The file was modifiedzorg/jenkins/monorepo_build.py (diff)
The file was modifiedtest/jenkins/test_monorepo_build.py (diff)

Summary

  1. [ARM][MachineOutliner] Remove unneeded dynamic allocation. (details)
  2. [FileCheck] Implement equality operators for ExpressionValue. (details)
  3. [ObjectYAML] - Remove unused function. NFC. (details)
  4. [Test] Add test showing missing opportunity of folding ICmp(Phi(Consts...)) (details)
  5. [InstCombine][NFC] Factor out constant check (details)
  6. Add support for IntegerLiteral in SyntaxTree (details)
  7. [gn build] port some -Wno flags for gcc from the cmake build. (details)
  8. [ARM] FP16 conversion tests. NFC (details)
  9. [VP][Fix] canIgnoreVectorLength for scalable types (details)
  10. [PhaseOrdering] add more tests for vector reductions; NFC (details)
  11. [analyzer] Ignore calculated indices of <= 0 in VLASizeChecker (details)
  12. [OpenCL] Add cl_khr_extended_subgroup extensions. (details)
  13. [x86] add FileCheck / assertions to test; NFC (details)
  14. [openmp] Fixed nonmonotonic schedule implementation. (details)
  15. [Support] Use outs() in ToolOutputFile (details)
  16. [gn build] Port 48cd9d9dd86 (details)
  17. [TableGen] Handle (outs variable_ops) (details)
  18. [x86] add test/code comment for chain value use (PR46195); NFC (details)
  19. [OPENMP]Fix PR45383: type dependent array subscripts are diagnosed erroneously. (details)
  20. RegAllocFast: Remove dead code (details)
Commit 6b9e1022436336b4ee7fa425f641f1cdcbd8f839 by yvan.roux
[ARM][MachineOutliner] Remove unneeded dynamic allocation.
The file was modifiedllvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
Commit ed9df8621a85329d9cb607e982a6efb2d9c670f5 by paul.walker
[FileCheck] Implement equality operators for ExpressionValue.

Subscribers: hiraditya, thopre, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D81094
The file was modifiedllvm/lib/Support/FileCheckImpl.h
The file was modifiedllvm/unittests/Support/FileCheckTest.cpp
Commit 9d739a915730493d3578ad73df4e86e841a0c08b by grimar
[ObjectYAML] - Remove unused function. NFC.

Was introduced in D81005 by mistake.

Catched by BB:
http://lab.llvm.org:8011/builders/clang-ppc64le-rhel/builds/4070/steps/build%20stage%201/logs/stdio
The file was modifiedllvm/lib/ObjectYAML/ELFEmitter.cpp
Commit 18134511d93046b7cde5c3cb9d5e356bd8e06b00 by mkazantsev
[Test] Add test showing missing opportunity of folding ICmp(Phi(Consts...))
The file was addedllvm/test/Transforms/InstCombine/icmp-constant-phi.ll
Commit 9bdb91889020b3e61cba26adb1b9c64a24c09f95 by mkazantsev
[InstCombine][NFC] Factor out constant check

We plan to add more transforms here. Besides, this check should be
done in the beginning just from function's name.
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
Commit 3b739690b01e595f6da412316b58a0a291702049 by gribozavr
Add support for IntegerLiteral in SyntaxTree

Reviewers: gribozavr2

Reviewed By: gribozavr2

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D81135
The file was modifiedclang/lib/Tooling/Syntax/Nodes.cpp
The file was modifiedclang/include/clang/Tooling/Syntax/Nodes.h
The file was modifiedclang/unittests/Tooling/Syntax/TreeTest.cpp
The file was modifiedclang/lib/Tooling/Syntax/BuildTree.cpp
Commit 3871ad366b11710afc0f8c1397ec6ffc4be64736 by thakis
[gn build] port some -Wno flags for gcc from the cmake build.
The file was modifiedllvm/utils/gn/build/BUILD.gn
Commit 7d4ebc98afac1e8f749644589eae6b26ddd68811 by david.green
[ARM] FP16 conversion tests. NFC
The file was addedllvm/test/CodeGen/Thumb2/mve-vcvt16.ll
The file was modifiedllvm/test/CodeGen/Thumb2/mve-shuffleext.ll
The file was addedllvm/test/CodeGen/Thumb2/mve-fp16convertloops.ll
Commit a0dfdda4e5e8c3f33ccc80f289374a60943cb7c3 by simon.moll
[VP][Fix] canIgnoreVectorLength for scalable types

This patch fixes VPIntrinsic::canIgnoreVectorLength when used on a
VPIntrinsic with scalable vector types. Also includes new unittest cases
for the '<vscale x 1 x whatever>' and '%evl == vscale' corner cases.
The file was modifiedllvm/lib/IR/IntrinsicInst.cpp
The file was modifiedllvm/unittests/IR/VPIntrinsicTest.cpp
Commit ecbf34c0e480654206330e53d8a103e824329083 by spatel
[PhaseOrdering] add more tests for vector reductions; NFC

More coverage for D80867.
The file was addedllvm/test/Transforms/PhaseOrdering/X86/vector-reductions-expanded.ll
Commit bd425825411af1b340134b0e8c9c03733ee87d66 by vince.a.bridgers
[analyzer] Ignore calculated indices of <= 0 in VLASizeChecker

Summary:
See https://bugs.llvm.org/show_bug.cgi?id=46128. The checker does not
yet comprehend constraints involving multiple symbols, so it's possible
to calculate a VLA size that's negative or 0. A LIT is added to catch
regressions, and this change simply bails if a VLA size of 0 or less is
calculated.

Reviewers: balazske, NoQ, martong, baloghadamsoftware, Szelethus, gamesh411

Reviewed By: balazske, NoQ, Szelethus

Subscribers: xazax.hun, szepet, rnkovacs, a.sidorin, mikhail.ramalho, donat.nagy, Charusso, ASDenysPetrov, cfe-commits, dkrupp

Tags: #clang

Differential Revision: https://reviews.llvm.org/D80903
The file was modifiedclang/lib/StaticAnalyzer/Checkers/VLASizeChecker.cpp
The file was modifiedclang/test/Analysis/vla.c
Commit 4a4402f0d72167477a6252e4c3daf5089ebc8f9a by anastasia.stulova
[OpenCL] Add cl_khr_extended_subgroup extensions.

Added extensions and their function declarations into
the standard header.

Patch by Piotr Fusik!

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79781
The file was modifiedclang/test/SemaOpenCL/extension-version.cl
The file was modifiedclang/lib/Headers/opencl-c.h
The file was modifiedclang/include/clang/Basic/OpenCLExtensions.def
Commit de38e882bb6510e8bf1685f474cd52adb4d145e5 by spatel
[x86] add FileCheck / assertions to test; NFC
The file was modifiedllvm/test/CodeGen/X86/membarrier.ll
Commit abe64360ae133e9b07c0a39cc54d09bd5fcb86f9 by Andrey.Churbanov
[openmp] Fixed nonmonotonic schedule implementation.

Differential Revision: https://reviews.llvm.org/D80942
The file was modifiedopenmp/runtime/src/kmp.h
The file was modifiedopenmp/runtime/src/kmp_dispatch.cpp
The file was modifiedopenmp/runtime/src/kmp_dispatch.h
The file was addedopenmp/runtime/test/worksharing/for/omp_nonmonotonic_nowait.c
Commit 48cd9d9dd86c2c37e6c47cbb23c06d36a1870b83 by pavel
[Support] Use outs() in ToolOutputFile

Summary:
If the output filename was specified as "-", the ToolOutputFile class
would create a brand new raw_ostream object referring to the stdout.
This patch changes it to reuse the llvm::outs() singleton.

At the moment, this change should be "NFC", but it does enable other
enhancements, like the automatic stdout/stderr synchronization as
discussed on D80803.

I've checked the history, and I did not find any indication that this
class *has* to use a brand new stream object instead of outs() --
indeed, it is special-casing "-" in a number of places already, so this
change fits the pattern pretty well. I suspect the main reason for the
current state of affairs is that the class was originally introduced
(r111595, in 2010) as a raw_fd_ostream subclass, which made any other
solution impossible.

Another potential benefit of this patch is that it makes it possible to
move the raw_ostream class out of the business of special-casing "-" for
stdout handling. That state of affairs does not seem appropriate because
"-" is a valid filename (albeit hard to access with a lot of command
line tools) on most systems. Handling "-" in ToolOutputFile seems more
appropriate.

To make this possible, this patch changes the return type of
llvm::outs() and errs() to raw_fd_ostream&. Previously the functions
were constructing objects of that type, but returning a generic
raw_ostream reference. This makes it possible for new ToolOutputFile and
other code to use raw_fd_ostream methods like error() on the outs()
object. This does not seem like a bad thing (since stdout is a file
descriptor which can be redirected to anywhere, it makes sense to ask it
whether the writing was successful or if it supports seeking), and
indeed a lot of code was already depending on this fact via the
ToolOutputFile "back door".

Reviewers: dblaikie, JDevlieghere, MaskRay, jhenderson

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D81078
The file was modifiedllvm/lib/Support/ToolOutputFile.cpp
The file was modifiedllvm/unittests/Support/CMakeLists.txt
The file was modifiedllvm/include/llvm/Support/ToolOutputFile.h
The file was addedllvm/unittests/Support/ToolOutputFileTest.cpp
The file was modifiedllvm/include/llvm/Support/raw_ostream.h
The file was modifiedmlir/include/mlir/Pass/PassManager.h
The file was modifiedllvm/lib/Support/raw_ostream.cpp
Commit 8ccb9f98ea98d0a5db179c10431fc25b9a20a1c6 by llvmgnsyncbot
[gn build] Port 48cd9d9dd86
The file was modifiedllvm/utils/gn/secondary/llvm/unittests/Support/BUILD.gn
Commit f2c97656644e783622a6e60fe452b41ffe0f1d18 by dantrushin
[TableGen] Handle (outs variable_ops)

When `variable_ops` is specified in `InOperandList` of instruction,
it behaves as expected, i.e., does not count as operand.
So for `(ins variable_ops)` instruction description will have 0
operands.  However when used in OutOperandList it is counted as
operand. So `(outs variable_ops)` results in instruction with
one def.
This patch makes behavior of `variable_ops` in `out` list to match
that of `in` list.

Reviewed By: reames
Differential Revision: https://reviews.llvm.org/D81095
The file was modifiedllvm/utils/TableGen/CodeGenInstruction.cpp
Commit 652b3757c848770885c1dc8f55292b427de1efa2 by spatel
[x86] add test/code comment for chain value use (PR46195); NFC
The file was modifiedllvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
The file was addedllvm/test/CodeGen/X86/load-chain.ll
Commit 3c191ae555cce6896f40b75ca031eadf675c5fb8 by a.bataev
[OPENMP]Fix PR45383: type dependent array subscripts are diagnosed erroneously.

Summary:
If the array subscript expression is type depent, its analysis must be
delayed before its instantiation.

Reviewers: jdoerfert

Subscribers: yaxunl, guansong, caomhin, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D78637
The file was modifiedclang/lib/Sema/SemaOpenMP.cpp
The file was modifiedclang/test/OpenMP/task_ast_print.cpp
Commit 931a68f26b9a3de853807ffad7b2cd0a2dd30922 by Matthew.Arsenault
RegAllocFast: Remove dead code
The file was modifiedllvm/lib/CodeGen/RegAllocFast.cpp