FailedChanges

Summary

  1. [lldb/DWARF] Always construct a DWARFDebugInfo object (details)
  2. [compiler-rt] [builtins] Fix logb / logbl tests (details)
  3. [NFC] Remove ar/ranlib test noise during cmake step (details)
  4. [ARM,MVE] Add vqdmull[b,t]q intrinsic families (details)
  5. [clang-tidy] misc-no-recursion: point to the function defs, not decls (details)
  6. SpeculativeExecution: fixed ingoring free execution (details)
  7. [AMDGPU] simplifyI24 - replace GetDemandedBits with SimplifyMultipleUseDemandedBits (details)
Commit ddf60ba09f2cf31851ea165c7c5fcfc4c2ad546e by pavel
[lldb/DWARF] Always construct a DWARFDebugInfo object

Change the return value of SymbolFileDWARF::DebugInfo from a pointer to
a reference, and remove all null checks.

Previously, we were not constructing the DebugInfo object when the
debug_info section was empty. Now we always construct the object but
it will return an empty list of dwarf units (a thing which it already
supported).
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDwo.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.h
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DebugNamesDWARFIndex.h
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DebugNamesDWARFIndex.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.h
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
Commit 689f1e85ba68db7440451a6834095a52d1526b00 by luismarques
[compiler-rt] [builtins] Fix logb / logbl tests

Like was done before in D67999 for `logbf`, this patch fixes the tests for
the internal compiler-rt implementations of `logb` and `logbl` to consider
all NaNs equivalent. Not doing so was resulting in test failures for
riscv64, since the the NaNs had different signs, but the spec doesn't
specify the NaN signedness or payload.

Fixes bug 44244.

Reviewers: rupprecht, delcypher
Reviewed By: rupprecht, delcypher
Differential Revision: https://reviews.llvm.org/D74826
The file was modifiedcompiler-rt/test/builtins/Unit/compiler_rt_logb_test.c
The file was modifiedcompiler-rt/test/builtins/Unit/compiler_rt_logbl_test.c
Commit def62c7f6548104274f95154bb5a91264646c701 by sguelton
[NFC] Remove ar/ranlib test noise during cmake step

At least on RHEL, ar outputs on stderr a message similar to

        .../bin/ar: creating t.a

Which creates noise during the cmake step. Get rid of it.
The file was modifiedllvm/cmake/modules/HandleLLVMOptions.cmake
Commit f4fd7dbf85e278eff303514760bff4773a87e601 by mikhail.maltsev
[ARM,MVE] Add vqdmull[b,t]q intrinsic families

Summary:
This patch adds two families of ACLE intrinsics: vqdmullbq and
vqdmulltq (including vector-vector and vector-scalar variants) and the
corresponding LLVM IR intrinsics llvm.arm.mve.vqdmull and
llvm.arm.mve.vqdmull.predicated.

Reviewers: simon_tatham, MarkMurrayARM, dmgreen, ostannard

Reviewed By: MarkMurrayARM

Subscribers: kristof.beyls, hiraditya, cfe-commits, llvm-commits

Tags: #clang, #llvm

Differential Revision: https://reviews.llvm.org/D74845
The file was modifiedllvm/lib/Target/ARM/ARMInstrMVE.td
The file was modifiedllvm/include/llvm/IR/IntrinsicsARM.td
The file was addedclang/test/CodeGen/arm-mve-intrinsics/vqdmulltq.c
The file was addedclang/test/CodeGen/arm-mve-intrinsics/vqdmullbq.c
The file was addedllvm/test/CodeGen/Thumb2/mve-intrinsics/vqdmull.ll
The file was modifiedclang/include/clang/Basic/arm_mve.td
Commit c8f9e526bc5319f2d0beed4012fbf17b11e15161 by lebedev.ri
[clang-tidy] misc-no-recursion: point to the function defs, not decls

Results in slightly better UX.
This actually was the initial intent, but it kinda got lost along the way.
The file was modifiedclang/include/clang/Analysis/CallGraph.h
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/misc-no-recursion.cpp
The file was modifiedclang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp
Commit dbfc682e2ba2581e214fabb04ba1b555ff8b274a by dfukalov
SpeculativeExecution: fixed ingoring free execution

Summary:
After updating cost model in AMDGPU target (47a5c36b37f0) the pass started to
ignore some BBs since they got all instructions estimated as free.

Reviewers: arsenm, chandlerc, nhaehnle

Reviewed By: nhaehnle

Subscribers: jvesely, wdng, nhaehnle, tpr, hiraditya, kerbowa, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74825
The file was addedllvm/test/CodeGen/AMDGPU/speculative-execution-freecasts.ll
The file was modifiedllvm/lib/Transforms/Scalar/SpeculativeExecution.cpp
Commit 6085593c128e91fd7db998c5441ebe120c7e4f04 by llvm-dev
[AMDGPU] simplifyI24 - replace GetDemandedBits with SimplifyMultipleUseDemandedBits

GetDemandedBits mostly just calls SimplifyMultipleUseDemandedBits now, but it does a very blunt constant simplification that SimplifyMultipleUseDemandedBits avoids.

If we need to demand bits from constants we should handle this through ShrinkDemandedConstant/targetShrinkDemandedConstant.

@arsenm confirmed that the sign extended immediates are better for code size.

Differential Revision: https://reviews.llvm.org/D74857
The file was modifiedllvm/test/CodeGen/AMDGPU/amdgpu-mul24-knownbits.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/lshl64-to-32.ll
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp