SuccessChanges

Summary

  1. [lldb win buildbot] Add a couple people to the default notifications (details)
Commit 3a4296e9c1cdb53e0bf939b244790d257a6d5f26 by stilis
[lldb win buildbot] Add a couple people to the default notifications
The file was modifiedbuildbot/osuosl/master/config/status.py (diff)

Summary

  1. [mlir][Parser] Use APFloat instead of FloatAttr when parsing DenseElementsAttrs. (details)
  2. Revert "[PatternMatch] Match XOR variant of unsigned-add overflow check." (details)
  3. [mlir][NFC] Fix warning for mismatched sign comparison. (details)
  4. [DDG] Data Dependence Graph - Graph Simplification (details)
  5. [mlir][NFC] Fix 'gatherLoops' utility (details)
  6. [CMake] Only detect the linker once in AddLLVM.cmake (details)
  7. [UpdateTestChecks] Add support for '.' in ir function names (details)
  8. [AMDGPU] Regenerate immediate constant tests (details)
  9. [libFuzzer] Fix denominator in the "inputs have DFT" log line. (details)
  10. Include static prof data when collecting loop BBs (details)
  11. [mlir] [VectorOps] Framework for progressive lowering of vector.contract (details)
  12. Reapply [IRBuilder] Always respect inserter/folder (details)
Commit 4a7364f1c2ef0c45d7e603799fe0b7662d4c4078 by riddleriver
[mlir][Parser] Use APFloat instead of FloatAttr when parsing DenseElementsAttrs.

Summary: DenseElementsAttr stores float values as raw bits internally, so creating attributes just to have them unwrapped is extremely inefficient.

Differential Revision: https://reviews.llvm.org/D74818
The file was modifiedmlir/lib/Parser/Parser.cpp
Commit c7fc0e5da6c3c36eb5f3a874a6cdeaedb26856e0 by flo
Revert "[PatternMatch] Match XOR variant of unsigned-add overflow check."

This reverts commit e01a3d49c224d6f8a7afc01205b05b9deaa07afa.
and commit a6a585b8030b6e8d4c50c71f54a6addb21995fe0.

This causes a failure on GreenDragon:
http://lab.llvm.org:8080/green/view/LLDB/job/lldb-cmake/9597
The file was modifiedllvm/test/CodeGen/X86/sat-add.ll
The file was modifiedllvm/test/Transforms/CodeGenPrepare/AArch64/overflow-intrinsics.ll
The file was modifiedllvm/lib/CodeGen/CodeGenPrepare.cpp
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineCompares.cpp
The file was modifiedllvm/test/Transforms/CodeGenPrepare/X86/overflow-intrinsics.ll
The file was modifiedllvm/include/llvm/IR/PatternMatch.h
The file was modifiedllvm/test/CodeGen/AArch64/sat-add.ll
Commit fd0e8b4c0ca78880bed974a5a14b7cd16a89d91c by riddleriver
[mlir][NFC] Fix warning for mismatched sign comparison.
The file was modifiedmlir/include/mlir/IR/OperationSupport.h
Commit 0a2626d0cdc2942799026c616fcac974e0a1e204 by bmahjour
[DDG] Data Dependence Graph - Graph Simplification

Summary:
This is the last functional patch affecting the representation of DDG.
Here we try to simplify the DDG to reduce the number of nodes and edges by
iteratively merging pairs of nodes that satisfy the following conditions,
until no such pair can be identified. A pair of nodes consisting of a and b
can be merged if:

    1. the only edge from a is a def-use edge to b and
    2. the only edge to b is a def-use edge from a and
    3. there is no cyclic edge from b to a and
    4. all instructions in a and b belong to the same basic block and
    5. both a and b are simple (single or multi instruction) nodes.

These criteria allow us to fold many uninteresting def-use edges that
commonly exist in the graph while avoiding the risk of introducing
dependencies that didn't exist before.

Authored By: bmahjour

Reviewer: Meinersbur, fhahn, myhsu, xtian, dmgreen, kbarton, jdoerfert

Reviewed By: Meinersbur

Subscribers: ychen, arphaman, simoll, a.elovikov, mgorny, hiraditya, jfb, wuzish, llvm-commits, jsji, Whitney, etiotto, ppc-slack

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D72350
The file was modifiedllvm/test/Analysis/DDG/basic-loopnest.ll
The file was modifiedllvm/include/llvm/Analysis/DDG.h
The file was modifiedllvm/lib/Analysis/DependenceGraphBuilder.cpp
The file was modifiedllvm/test/Analysis/DDG/basic-b.ll
The file was modifiedllvm/test/Analysis/DDG/basic-a.ll
The file was modifiedllvm/include/llvm/Analysis/DependenceGraphBuilder.h
The file was modifiedllvm/lib/Analysis/DDG.cpp
Commit 376c68539c5a4b8baf7dc8f34ffa71daadada28f by diego.caballero
[mlir][NFC] Fix 'gatherLoops' utility

It replaces DenseMap output with a SmallVector and it
removes empty loop levels from the output.

Reviewed By: andydavis1, mehdi_amini

Differential Revision: https://reviews.llvm.org/D74658
The file was modifiedmlir/lib/Transforms/Utils/LoopUtils.cpp
The file was modifiedmlir/test/lib/Transforms/TestAffineDataCopy.cpp
The file was modifiedmlir/test/lib/Transforms/TestLoopFusion.cpp
The file was modifiedmlir/include/mlir/Transforms/LoopUtils.h
Commit 84f80b1faaeb385a178f4b517d92d29ca6a6d854 by Louis Dionne
[CMake] Only detect the linker once in AddLLVM.cmake

Summary:
Otherwise, the build output contains a bunch of "Linker detection: <xxx>"
lines that are really redundant. We also make redundant calls to the
linker, although that is a smaller concern.

Reviewers: smeenai

Subscribers: mgorny, fedor.sergeev, jkorous, dexonsmith, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D68648
The file was modifiedllvm/cmake/modules/AddLLVM.cmake
Commit 59982a6d8f9d0ba34a47798fe155121767e66411 by llvm-dev
[UpdateTestChecks] Add support for '.' in ir function names

Will let us regenerate from amdgpu float constant tests
The file was modifiedllvm/utils/UpdateTestChecks/common.py
Commit 025ff5a4eaf7dd01810e0dd0deb38a204a32a357 by llvm-dev
[AMDGPU] Regenerate immediate constant tests
The file was modifiedllvm/test/CodeGen/AMDGPU/imm.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/imm16.ll
Commit 076475713c236081a3247a53e9dbab9043c3eac2 by mmoroz
[libFuzzer] Fix denominator in the "inputs have DFT" log line.

Summary:
The number of "inputs have the Data Flow Trace" cannot be greater than
the number of inputs touching the focus function. The existing message is rather
confusing as the same log would mention a greater total number of traces a few
lines above.

Reviewers: kcc, metzman

Subscribers: #sanitizers, llvm-commits

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D74779
The file was modifiedcompiler-rt/test/fuzzer/dataflow.test
The file was modifiedcompiler-rt/lib/fuzzer/FuzzerLoop.cpp
Commit 129c911efaa492790c251b3eb18e4db36b55cbc5 by isanbard
Include static prof data when collecting loop BBs

Summary:
If the programmer adds static profile data to a branch---i.e. uses
"__builtin_expect()" or similar---then we should honor it. Otherwise,
"__builtin_expect()" is ignored in crucial situations. So we trust that
the programmer knows what they're doing until proven wrong.

Subscribers: hiraditya, JDevlieghere, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74809
The file was modifiedllvm/lib/CodeGen/MachineBlockPlacement.cpp
The file was modifiedllvm/test/CodeGen/X86/move_latch_to_loop_top.ll
The file was modifiedllvm/test/CodeGen/X86/ragreedy-bug.ll
The file was modifiedllvm/test/CodeGen/Hexagon/prof-early-if.ll
The file was modifiedllvm/include/llvm/CodeGen/MachineLoopInfo.h
The file was modifiedllvm/lib/CodeGen/MachineLoopInfo.cpp
The file was modifiedllvm/test/CodeGen/X86/block-placement.ll
The file was addedllvm/test/CodeGen/X86/block-placement-2.ll
Commit 0ba9ee9f0e55b1f9155a5bb5739550860fa5fec2 by ajcbik
[mlir] [VectorOps] Framework for progressive lowering of vector.contract

Summary:
Lowers all free/batch dimensions in a vector.contract progressively
into simpler vector.contract operations until a direct vector.reduction
operation is reached. Then lowers 1-D reductions into vector.reduce.

Still TBD:
multi-dimensional contractions that remain after removing all the parallel dims

Reviewers: nicolasvasilache, andydavis1, rriddle

Reviewed By: andydavis1

Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74797
The file was modifiedmlir/test/Dialect/VectorOps/vector-contract-transforms.mlir
The file was modifiedmlir/lib/Dialect/VectorOps/VectorTransforms.cpp
Commit f6875c434ec20eb4f24495317592f64334347784 by nikita.ppv
Reapply [IRBuilder] Always respect inserter/folder

Some IRBuilder methods that were originally defined on
IRBuilderBase do not respect custom IRBuilder inserters/folders,
because those were not accessible prior to D73835. Fix this by
making use of existing (and now accessible) IRBuilder methods,
which will handle inserters/folders correctly.

There are some changes in OpenMP and Instrumentation tests, where
bitcasts now get constant folded. I've also highlighted one
InstCombine test which now finishes in two rather than three
iterations, thanks to new instructions being inserted into the
worklist.

Differential Revision: https://reviews.llvm.org/D74787
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/Mips/vararg-mips64el.ll
The file was modifiedllvm/lib/IR/IRBuilder.cpp
The file was modifiedllvm/test/Transforms/InstCombine/saturating-add-sub.ll
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/Mips/vararg-mips64.ll
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/msan_basic.ll
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/X86/vararg_call.ll
The file was modifiedclang/test/OpenMP/parallel_codegen.cpp
The file was modifiedclang/test/OpenMP/target_firstprivate_codegen.cpp
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/PowerPC/vararg-ppc64le.ll
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/byval-alignment.ll
The file was modifiedllvm/test/Instrumentation/MemorySanitizer/PowerPC/vararg-ppc64.ll