SuccessChanges

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

Summary

  1. [analyzer] SATestBuild.py: Allow comments in run_static_analyzer.cmd. (details)
  2. [lldb] Revive TestBasicEntryValuesX86_64 (details)
  3. [lldb/test] Make "inline" tests handle multiple statements at the same location (details)
  4. [x86] add test for funnel shift in loop with cross-block splat variable; NFC (details)
  5. Revert "[DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression" (details)
  6. [mlir][Linalg] NFC - Refactor and simplify Promotion (details)
  7. [Clang] Fix the incorrect return type of atomic_is_lock_free (details)
  8. [PowerPC] Use PredictableSelectIsExpensive to enable select to branch in CGP (details)
  9. [Attributor] Fix for a crash on RAUW when rewriting function signature (details)
  10. [lldb] Speculative fix for the entry values test on arm64 (details)
Commit ec2d93c7d7b00374535cb3d6658abec4b150f6f6 by Artem Dergachev
[analyzer] SATestBuild.py: Allow comments in run_static_analyzer.cmd.

Because those can get really weird sometimes.
The file was modifiedclang/utils/analyzer/SATestBuild.py
Commit a5c23179826a6dc4f26a4f3fcda66290bc04a201 by pavel
[lldb] Revive TestBasicEntryValuesX86_64

Summary:
This function rewrites the test to be (hopefully) less susceptible to
codegen changes and re-enables it.

The most interesting changes are:
- use an __attribute__((optnone)) function instead of a volatile asm to
  "use" a value. This isn't strictly necessary, but it makes the
  function simpler while achieving the same effect.
- use a call to a function with the exact same signature instead of a
  volatile asm to "destroy" arguments. This makes the independent of the
  ABI, and (together with avoiding the usage of the arguments after the
  call) ensures that the compiler has no reason to move the argument
  from its initial register (previously we needed to guess where will
  the compiler store the arguments).

Reviewers: vsk, djtodoro, dblaikie

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D79491
The file was addedlldb/test/API/functionalities/param_entry_vals/basic_entry_values/Makefile
The file was removedlldb/test/API/functionalities/param_entry_vals/basic_entry_values_x86_64/TestBasicEntryValuesX86_64.py
The file was addedlldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp
The file was removedlldb/test/API/functionalities/param_entry_vals/basic_entry_values_x86_64/Makefile
The file was addedlldb/test/API/functionalities/param_entry_vals/basic_entry_values/TestBasicEntryValues.py
The file was removedlldb/test/API/functionalities/param_entry_vals/basic_entry_values_x86_64/main.cpp
Commit ba898282bc204f78016c2e7779dc5cde37e10fd1 by pavel
[lldb/test] Make "inline" tests handle multiple statements at the same location

Summary:
The test machinery translates each continuous block of "//%" comments
into a single breakpoint. If there's no code between the blocks the
breakpoints will end up at the same location in the program. When the
process stops at a breakpoint lldb correctly reports all breakpoint IDs,
but the test machinery only looks at the first one. This results in a
very dangerous situation as it means some checks can be silently
stopped.

This patch fixes that by making the test machinery iterate through all
breakpoints at a given location and execute all commands.

Reviewers: vsk, JDevlieghere

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D79563
The file was modifiedlldb/test/API/commands/expression/dollar-in-variable/main.c
The file was modifiedlldb/packages/Python/lldbsuite/test/lldbinline.py
Commit b75795c25caa63d97849263476178c8b44ebe9dc by spatel
[x86] add test for funnel shift in loop with cross-block splat variable; NFC
The file was modifiedllvm/test/CodeGen/X86/vector-fshl-128.ll
Commit 728cf6d86b4f20144ac10517afb0cb978beac124 by sam.mccall
Revert "[DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression"

This reverts commit 3c44c441db0f8d7e210806b5b221cd9ed66f2d7b.

Causes infloops on some inputs, see https://reviews.llvm.org/D77319 for repro
The file was modifiedllvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
The file was modifiedllvm/test/CodeGen/X86/neg_fp.ll
The file was modifiedllvm/include/llvm/CodeGen/TargetLowering.h
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUISelLowering.h
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.h
Commit 8dbbb223834d1715bc9869aa409a4b0f52816da3 by ntv
[mlir][Linalg] NFC - Refactor and simplify Promotion

Summary: This revision introduces LinalgPromotionOptions to more easily control the application of promotion patterns. It also simplifies the different entry points into Promotion in preparation for some behavior change in subsequent revisions.

Differential Revision: https://reviews.llvm.org/D79489
The file was modifiedmlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h
The file was modifiedmlir/lib/Dialect/Linalg/Transforms/Promotion.cpp
The file was modifiedmlir/test/lib/Transforms/TestLinalgTransforms.cpp
The file was modifiedmlir/lib/Dialect/Linalg/Transforms/Transforms.cpp
Commit 9aee35bcc90faa9db3ea0111c0a80ebee7446cac by Louis Dionne
[Clang] Fix the incorrect return type of atomic_is_lock_free

Fixing the return type of atomic_is_lock_free as per
https://en.cppreference.com/w/c/atomic/atomic_is_lock_free

Differential Revision: https://reviews.llvm.org/D79504
The file was modifiedclang/test/CodeGen/atomic-ops.c
The file was modifiedclang/include/clang/Basic/Builtins.def
The file was modifiedclang/test/CodeGen/big-atomic-ops.c
Commit dcc5ff3bc2e85a07ff2192223c652fa2cdb164ef by shkzhang
[PowerPC] Use PredictableSelectIsExpensive to enable select to branch in CGP

Summary:
This patch will set the variable PredictableSelectIsExpensive to do the
select to if based on BranchProbability in CodeGenPrepare.

When the BranchProbability more than MinPercentageForPredictableBranch,
PPC will convert SELECT to branch.

Reviewed By: nemanjai

Differential Revision: https://reviews.llvm.org/D71883
The file was modifiedllvm/lib/Target/PowerPC/PPCSubtarget.h
The file was modifiedllvm/lib/Target/PowerPC/PPC.td
The file was modifiedllvm/lib/Target/PowerPC/PPCSubtarget.cpp
The file was addedllvm/test/CodeGen/PowerPC/select-to-branch.mir
The file was modifiedllvm/lib/Target/PowerPC/PPCISelLowering.cpp
Commit 3df40007e6317955176dff0fa9e0b029dc4a11d1 by serguei.n.dmitriev
[Attributor] Fix for a crash on RAUW when rewriting function signature

Reviewers: jdoerfert, sstefan1, uenoku

Reviewed By: uenoku

Subscribers: hiraditya, uenoku, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D79680
The file was removedllvm/test/Transforms/Attributor/returned_crash.ll
The file was addedllvm/test/Transforms/Attributor/misc_crash.ll
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
Commit caa2408cbe0a21e175cb287795d283ec4196cdee by pavel
[lldb] Speculative fix for the entry values test on arm64

The bot fails with a message which seems to indicate a problem in option
parsing. Avoid grouping the options to see if that helps.
The file was modifiedlldb/test/API/functionalities/param_entry_vals/basic_entry_values/main.cpp