SuccessChanges

Summary

  1. [ThinLTO] Pass CodeGenOpts like UnrollLoops/VectorizeLoop/VectorizeSLP (details)
  2. [xray] Remove cl::sub from alias options (details)
  3. clang-tidy doc - remove the widths (details)
  4. [Syntax] Update comment, remove stale FIXME. NFC (details)
  5. [LV] VPValues for memory operation pointers (NFCI) (details)
  6. Restore order in clang-tidy section of release notes (details)
  7. Allow system header to provide their own implementation of some builtin (details)
  8. [NFC] [PowerPC] Add isPredicable for basic instrs (details)
  9. [clangd] Improve type printing in hover (details)
  10. [clangd] Fix markdown rendering in VSCode (details)
  11. Fix several issues with compiler extensions (details)
  12. [SVEV] Recognise hardware-loop intrinsic loop.decrement.reg (details)
  13. Follow up of 67bf9a6154d4b82c, minor fix in test case, removed duplicate (details)
  14. [libcxx] Force-cache LIBCXX_CXX_ABI_LIBRARY_PATH (details)
  15. [CMake] Support running libc++abi tests in CrossWinToARMLinux cache file (details)
  16. XFAIL load_extension.ll test on macOS only (details)
  17. [MIR] Fix cyclic dependency of MIR formatter (details)
  18. [LV] Silence unused variable warning in Release builds. NFC. (details)
  19. Don't use dyn_cast_or_null if we know the pointer is nonnull. (details)
  20. [X86][AVX] Add tests for v8f32/v8i32 089abcde and 0189abcd shuffles (details)
  21. Fix Wdocumentation warning. NFCI. (details)
  22. Fix Wdocumentation warning. NFCI. (details)
  23. Data formatters: Look through array element typedefs (details)
  24. Fix "pointer is null" static analyzer warnings. NFCI. (details)
  25. [ARM,MVE] Make `vqrshrun` generate the right instruction. (details)
  26. [ARM][Thumb2] Fix ADD/SUB invalid writes to SP (details)
  27. [lldb/lua] Make convenience_variables.test compatible with lua-5.1 (details)
  28. [lldb/DWARF] Don't automatically search dwo unit attributes (details)
  29. [CodeComplete] Suggest 'return nullptr' in functions returning pointers (details)
  30. [lldb] Surpress "bitfield too small" gcc warning (details)
  31. [Support] ThreadPoolExecutor fixes for Windows/MinGW (details)
  32. [lldb][tests] Take into account all parent's categories when traverse (details)
  33. [lldb][test] NFC, re-use _getTestPath() function (details)
  34. [FPEnv] Generate constrained FP comparisons from clang (details)
  35. [lldb][tests] Cleanup '.categories' (details)
  36. Reverting, broke some bots. Need further investigation. (details)
  37. Sprinkle some constexpr on default ctors so the compiler can diagnose (details)
  38. ARMLowOverheadLoops: a few more dbg msgs to better trace rejected TP (details)
  39. RangeDataVector: Support custom sorting for D63540 (details)
  40. [ARM][MVE] Tail predicate VMAX,VMAXA,VMIN,VMINA (details)
Commit 21a4710c67a97838dd75cf60ed24da11280800f8 by wmi
[ThinLTO] Pass CodeGenOpts like UnrollLoops/VectorizeLoop/VectorizeSLP
down to pass builder in ltobackend.
Currently CodeGenOpts like UnrollLoops/VectorizeLoop/VectorizeSLP in
clang are not passed down to pass builder in ltobackend when new pass
manager is used. This is inconsistent with the behavior when new pass
manager is used and thinlto is not used. Such inconsistency causes slp
vectorization pass not being enabled in ltobackend for O3 + thinlto
right now. This patch fixes that.
Differential Revision: https://reviews.llvm.org/D72386
The file was addedclang/test/CodeGen/thinlto-slp-vectorize-pm.c
The file was modifiedlld/COFF/CMakeLists.txt
The file was modifiedllvm/include/llvm/LTO/Config.h
The file was addedllvm/test/tools/gold/X86/slp-vectorize-pm.ll
The file was modifiedllvm/lib/LTO/LTOBackend.cpp
The file was addedlld/test/ELF/lto/slp-vectorize-pm.ll
The file was modifiedllvm/tools/gold/gold-plugin.cpp
The file was modifiedllvm/tools/llvm-lto2/llvm-lto2.cpp
The file was modifiedlld/wasm/CMakeLists.txt
The file was modifiedlld/ELF/LTO.cpp
The file was modifiedclang/lib/CodeGen/BackendUtil.cpp
The file was modifiedllvm/test/Other/new-pm-thinlto-defaults.ll
The file was modifiedllvm/tools/llvm-lto2/CMakeLists.txt
The file was addedllvm/test/tools/llvm-lto2/X86/slp-vectorize-pm.ll
The file was modifiedllvm/lib/Passes/PassBuilder.cpp
The file was modifiedllvm/test/Other/new-pm-defaults.ll
The file was modifiedlld/ELF/CMakeLists.txt
Commit 995c18fc5051850782b1c096233867b8e56e0dea by smeenai
[xray] Remove cl::sub from alias options
Currently running the xray tools generates a number of errors:
$ ./bin/llvm-xray
: for the   -k option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -d option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -o option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -f option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -s option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -r option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -p option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
: for the   -m option: cl::alias must not have cl::sub(), aliased
option's cl::sub() will be used!
<snip>
Patch by Ryan Mansfield.
Differential Revision: https://reviews.llvm.org/D69386
The file was modifiedllvm/tools/llvm-xray/xray-account.cpp
The file was modifiedllvm/tools/llvm-xray/xray-graph.cpp
The file was modifiedllvm/tools/llvm-xray/xray-stacks.cpp
The file was modifiedllvm/tools/llvm-xray/xray-converter.cpp
The file was modifiedllvm/tools/llvm-xray/xray-extract.cpp
The file was modifiedllvm/tools/llvm-xray/xray-graph-diff.cpp
Commit b38d0d5bdb353f8496a0fc38b9bbee419b41a321 by sylvestre
clang-tidy doc - remove the widths
The file was modifiedclang-tools-extra/docs/clang-tidy/checks/list.rst
Commit 759c90456d418ffe69e1a2b4bcea2792491a6b5a by ibiryukov
[Syntax] Update comment, remove stale FIXME. NFC
The file was modifiedclang/include/clang/Tooling/Syntax/Tokens.h
Commit 8647a72c4a52e0386c0397ce3fbd38121c18b873 by gil.rapaport
[LV] VPValues for memory operation pointers (NFCI)
Memory instruction widening recipes use the pointer operand of their
load/store ingredient for generating the needed GEPs, making it
difficult to feed these recipes with pointers based on other ingredients
or none at all. This patch modifies these recipes to use a VPValue for
the pointer instead, in order to reduce ingredient def-use usage by ILV
as a step towards full VPlan-based def-use relations. The recipes are
constructed with VPValues bound to these ingredients, maintaining
current behavior.
Differential revision: https://reviews.llvm.org/D70865
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorize.cpp
The file was modifiedllvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
The file was modifiedllvm/lib/Transforms/Vectorize/VPlan.h
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
The file was modifiedllvm/lib/Transforms/Vectorize/VPlan.cpp
Commit 164da673009ba6c100ce45b6fa9a5dfd3b0b8e38 by hans
Restore order in clang-tidy section of release notes
Major changes are introduction of subsubsections to prevent people
putting new entries in wrong places. I also polished line length and
highlighting.
Patch by Eugene Zelenko!
The file was modifiedclang-tools-extra/docs/ReleaseNotes.rst
Commit 921f871ac438175ca8fcfcafdfcfac4d7ddf3905 by sguelton
Allow system header to provide their own implementation of some builtin
If a system header provides an (inline) implementation of some of their
function, clang still matches on the function name and generate the
appropriate llvm builtin, e.g. memcpy. This behavior is in line with
glibc recommendation « users may not provide their own version of
symbols » but doesn't account for the fact that glibc itself can provide
inline version of some functions.
It is the case for the memcpy function when -D_FORTIFY_SOURCE=1 is on.
In that case an inline version of memcpy calls __memcpy_chk, a function
that performs extra runtime checks. Clang currently ignores the inline
version and thus provides no runtime check.
This code fixes the issue by detecting functions whose name is a builtin
name but also have an inline implementation.
Differential Revision: https://reviews.llvm.org/D71082
The file was modifiedclang/lib/CodeGen/CGExpr.cpp
The file was modifiedclang/include/clang/AST/Decl.h
The file was addedclang/test/CodeGen/memcpy-nobuiltin.c
The file was modifiedclang/lib/AST/Decl.cpp
The file was addedclang/test/CodeGen/memcpy-nobuiltin.inc
The file was modifiedclang/lib/CodeGen/CodeGenModule.cpp
Commit 45c4b08d8228f64b02b8a4df069aa37d5fa70829 by qiucofan
[NFC] [PowerPC] Add isPredicable for basic instrs
PowerPC uses a dedicated method to check if the machine instr is
predicable by opcode. However, there's a bit `isPredicable` in instr
definition. This patch removes the method and set the bit only to
opcodes referenced in it.
Differential Revision: https://reviews.llvm.org/D71921
The file was modifiedllvm/lib/Target/PowerPC/PPCInstrInfo.h
The file was modifiedllvm/lib/Target/PowerPC/PPCInstrInfo.cpp
The file was modifiedllvm/lib/Target/PowerPC/PPCInstrInfo.td
The file was modifiedllvm/lib/Target/PowerPC/PPCInstr64Bit.td
Commit ffd0f116754c36146bb21a01b047782ce8a01e2e by kadircet
[clangd] Improve type printing in hover
Summary: Do not include tag keywords when printing types for symbol
names, as it will come from SymbolKind. Also suppress them while
printing definitions to prevent them occuring in template arguments.
Make use of `getAsString`, instead of `print` in all places to have a
consistent style across the file.
Reviewers: sammccall
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95,
cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72450
The file was modifiedclang-tools-extra/clangd/unittests/HoverTests.cpp
The file was modifiedclang-tools-extra/clangd/Hover.cpp
Commit abfa27e4f04dd84774bcfe15783942e21be391a5 by kadircet
[clangd] Fix markdown rendering in VSCode
Summary: Eventough it is OK to have a new line without any preceding
spaces in some markdown specifications, VSCode requires two spaces
before a new line to break a line inside a paragraph.
Reviewers: sammccall, ilya-biryukov
Subscribers: MaskRay, jkorous, arphaman, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72462
The file was modifiedclang-tools-extra/clangd/FormattedString.cpp
The file was modifiedclang-tools-extra/clangd/unittests/FormattedStringTests.cpp
Commit 346de9b67228f42eb9b55fa3b426b5dedfdb1d40 by sguelton
Fix several issues with compiler extensions
- Update documentation now that the move to monorepo has been made
- Do not tie compiler extension testing to LLVM_BUILD_EXAMPLES
- No need to specify LLVM libraries for plugins
- Add NO_MODULE option to match Polly specific requirements (i.e.
building the
module *and* linking it statically)
- Issue a warning when building the compiler extension with
LLVM_BYE_LINK_INTO_TOOLS=ON, as it modifies the behavior of clang,
which only
makes sense for testing purpose.
Still mark llvm/test/Feature/load_extension.ll as XFAIL because of a
ManagedStatic dependency that's going to be fixed in a seperate commit.
Differential Revision: https://reviews.llvm.org/D72327
The file was modifiedllvm/cmake/modules/AddLLVM.cmake
The file was modifiedllvm/examples/Bye/CMakeLists.txt
The file was modifiedllvm/test/lit.cfg.py
The file was modifiedpolly/lib/CMakeLists.txt
Commit 67bf9a6154d4b82c6c01aad01141bf08c1bbd0f6 by sjoerd.meijer
[SVEV] Recognise hardware-loop intrinsic loop.decrement.reg
Teach SCEV about the @loop.decrement.reg intrinsic, which has exactly
the same semantics as a sub expression. This allows us to query
hardware-loops, which contain this @loop.decrement.reg intrinsic, so
that we can calculate iteration counts, exit values, etc. of
hardwareloops.
This "int_loop_decrement_reg" intrinsic is defined as "IntrNoDuplicate".
Thus, while hardware-loops and tripcounts now become analysable by SCEV,
this prevents the usual loop transformations from applying
transformations on hardware-loops, which is what we want at this point,
for which I have added test cases for loopunrolling and IndVarSimplify
and LFTR.
Differential Revision: https://reviews.llvm.org/D71563
The file was addedllvm/test/Transforms/LoopUnroll/ARM/dont-unroll-loopdec.ll
The file was modifiedllvm/unittests/Analysis/ScalarEvolutionTest.cpp
The file was modifiedllvm/test/Transforms/IndVarSimplify/lftr.ll
The file was modifiedllvm/lib/Analysis/ScalarEvolution.cpp
Commit 356685a1d8972180f472c1333e8e89dbcc704c1d by sjoerd.meijer
Follow up of 67bf9a6154d4b82c, minor fix in test case, removed duplicate
option
The file was modifiedllvm/test/Transforms/LoopUnroll/ARM/dont-unroll-loopdec.ll
Commit 41f4dfd63ea0fe995ddfba1838aa5ed972cc1377 by jaskiewiczs
[libcxx] Force-cache LIBCXX_CXX_ABI_LIBRARY_PATH
Summary: The `LIBCXX_CXX_ABI_LIBRARY_PATH` CMake variable is cached once
in libcxx/cmake/Modules/HandleLibCXXABI.cmake in the `setup_abi_lib`
macro, and then cached again in libcxx/test/CMakeLists.txt. There, if it
is not set to a value, it is by default set to `LIBCXX_LIBRARY_DIR`.
However, this new value is not actually cached, because the old (empty)
value has been already cached. Use the `FORCE` CMake flag so that it is
saved to the cache.
This should not break anything, because the code changed here previously
had no effect, when it should have.
Reviewers: jroelofs, bcraig, ldionne, EricWF, mclow.lists, vvereschaka,
eastig
Reviewed By: vvereschaka
Subscribers: mgorny, christof, dexonsmith, libcxx-commits
Tags: #libc
Differential Revision: https://reviews.llvm.org/D69169
The file was modifiedlibcxx/test/CMakeLists.txt
Commit e44dedd3631c20bc4a1e62b68919a11168d39354 by jaskiewiczs
[CMake] Support running libc++abi tests in CrossWinToARMLinux cache file
Summary: Now that D71894 has landed, we're able to run libc++abi tests
remotely.
For that we can use the same CMake command as before. The tests can be
run using `ninja check-cxxabi`.
Reviewers: andreil99, vvereschaka, aorlov
Reviewed By: vvereschaka, aorlov
Subscribers: mgorny, kristof.beyls, ldionne, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72459
The file was modifiedclang/cmake/caches/CrossWinToARMLinux.cmake
Commit a1cc19b581443c84fff4c6e6d4e341351ef3203c by sguelton
XFAIL load_extension.ll test on macOS only
Other setup have been fixed by 346de9b67228f42eb9b55fa3b426b5dedfdb1d40
The file was modifiedllvm/test/Feature/load_extension.ll
Commit cfd849840134c4632c2f4fa498dfb93c47825b24 by benny.kra
[MIR] Fix cyclic dependency of MIR formatter
Summary: Move MIR formatter pointer from TargetMachine to
TargetInstrInfo to avoid cyclic dependency between target & codegen.
Reviewers: dsanders, bkramer, arsenm
Subscribers: wdng, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D72485
The file was modifiedllvm/lib/CodeGen/MIRPrinter.cpp
The file was modifiedllvm/lib/CodeGen/MachineInstr.cpp
The file was modifiedllvm/include/llvm/CodeGen/TargetInstrInfo.h
The file was modifiedllvm/lib/CodeGen/MIRParser/MIParser.cpp
The file was modifiedllvm/lib/CodeGen/SelectionDAG/SelectionDAGDumper.cpp
The file was modifiedllvm/include/llvm/Target/TargetMachine.h
The file was modifiedllvm/include/llvm/CodeGen/MachineMemOperand.h
The file was modifiedllvm/lib/CodeGen/MachineOperand.cpp
The file was modifiedllvm/lib/Target/TargetMachine.cpp
Commit 498856fca5b9306f545554aeec93c7c058f03eb3 by benny.kra
[LV] Silence unused variable warning in Release builds. NFC.
The file was modifiedllvm/lib/Transforms/Vectorize/LoopVectorize.cpp
Commit 2e66405d8d8ed818cb9310b6c33419bd8d803d96 by llvm-dev
Don't use dyn_cast_or_null if we know the pointer is nonnull.
Fix clang static analyzer null dereference warning by using dyn_cast
instead.
The file was modifiedllvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp
Commit 3804ac63d5d8443d0b6826b566e9cbb84d8898f3 by llvm-dev
[X86][AVX] Add tests for v8f32/v8i32 089abcde and 0189abcd shuffles
Mentioned in D66004
The file was modifiedllvm/test/CodeGen/X86/vector-shuffle-256-v8.ll
Commit b2cd273416f82b6c5efeb6138276d9e6b6f8256e by llvm-dev
Fix Wdocumentation warning. NFCI.
The file was modifiedllvm/lib/CodeGen/MachineOutliner.cpp
Commit f3849f739e52510871d11361125f0ef239f11603 by llvm-dev
Fix Wdocumentation warning. NFCI.
The file was modifiedclang/lib/Parse/ParseExpr.cpp
Commit 902974277d507a149e33487d32e4ba58c41451b6 by Raphael Isemann
Data formatters: Look through array element typedefs
Summary: Motivation: When formatting an array of typedefed chars, we
would like to display the array as a string.
The string formatter currently does not trigger because the formatter
lookup does not resolve typedefs for array elements (this behavior is
inconsistent with pointers, for those we do look through pointee
typedefs). This patch tries to make the array formatter lookup somewhat
consistent with the pointer formatter lookup.
Reviewers: teemperor, clayborg
Reviewed By: teemperor, clayborg
Subscribers: clayborg, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D72133
The file was modifiedlldb/source/DataFormatters/FormatManager.cpp
The file was modifiedlldb/source/API/SBType.cpp
The file was addedlldb/packages/Python/lldbsuite/test/functionalities/data-formatter/array_typedef/Makefile
The file was addedlldb/packages/Python/lldbsuite/test/functionalities/data-formatter/array_typedef/TestArrayTypedef.py
The file was addedlldb/packages/Python/lldbsuite/test/functionalities/data-formatter/array_typedef/main.cpp
The file was modifiedlldb/source/Symbol/ClangASTContext.cpp
Commit 870f6917936fdb8050be3ca3c67d9259390c4326 by llvm-dev
Fix "pointer is null" static analyzer warnings. NFCI.
Assert that the pointers are non-null before dereferencing them.
The file was modifiedllvm/lib/Target/Hexagon/HexagonEarlyIfConv.cpp
Commit 1ccee0e86386762bd742fd067391b6c4be089806 by simon.tatham
[ARM,MVE] Make `vqrshrun` generate the right instruction.
Summary: A copy-paste error in `arm_mve.td` meant that the MVE
`vqrshrun` intrinsic family was generating the `vqshrun` machine
instruction, because in the IR intrinsic call, the rounding flag
argument was set to 0 rather than 1.
Reviewers: dmgreen, MarkMurrayARM, miyuki, ostannard
Reviewed By: dmgreen
Subscribers: kristof.beyls, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72496
The file was modifiedclang/test/CodeGen/arm-mve-intrinsics/vector-shift-imm-dyadic.c
The file was modifiedclang/include/clang/Basic/arm_mve.td
Commit 8c12769f3046029e2a9b4e48e1645b1a77d28650 by diogo.sampaio
[ARM][Thumb2] Fix ADD/SUB invalid writes to SP
Summary: This patch fixes pr23772  [ARM] r226200 can emit illegal thumb2
instruction: "sub sp, r12, #80". The violation was that SUB and ADD
(reg, immediate) instructions can only write to SP if the source
register is also SP. So the above instructions was unpredictable. To
enforce that the instruction t2(ADD|SUB)ri does not write to SP we now
enforce the destination register to be rGPR (That exclude PC and SP).
Different than the ARM specification, that defines one instruction that
can read from SP, and one that can't, here we inserted one that can't
write to SP, and other that can only write to SP as to reuse most of the
hard-coded size optimizations. When performing this change, it uncovered
that emitting Thumb2 Reg plus Immediate could not emit all variants of
ADD SP, SP #imm instructions before so it was refactored to be able to.
(see test/CodeGen/Thumb2/mve-stacksplot.mir where we use a subw sp, sp,
Imm12 variant ) It also uncovered a disassembly issue of adr.w
instructions, that were only written as SUBW instructions (see
llvm/test/MC/Disassembler/ARM/thumb2.txt).
Reviewers: eli.friedman, dmgreen, carwil, olista01, efriedma
Reviewed By: efriedma
Subscribers: john.brawn, efriedma, ostannard, kristof.beyls, hiraditya,
dmgreen, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D70680
The file was modifiedllvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
The file was modifiedllvm/test/MC/ARM/thumb-diagnostics.s
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb2-v8.txt
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb-tests.txt
The file was addedllvm/test/CodeGen/Thumb2/bug-subw.ll
The file was modifiedllvm/lib/Target/ARM/Thumb2InstrInfo.cpp
The file was modifiedllvm/test/MC/ARM/negative-immediates.s
The file was modifiedllvm/test/CodeGen/MIR/ARM/thumb2-sub-sp-t3.mir
The file was modifiedllvm/test/MC/Disassembler/ARM/invalid-thumbv7.txt
The file was modifiedllvm/lib/Target/ARM/ARMAsmPrinter.cpp
The file was modifiedllvm/lib/Target/ARM/ARMInstrThumb2.td
The file was modifiedllvm/test/CodeGen/Thumb2/peephole-addsub.mir
The file was modifiedllvm/test/MC/ARM/basic-thumb2-instructions.s
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb2.txt
The file was modifiedllvm/test/CodeGen/Thumb2/fp16-stacksplot.mir
The file was modifiedllvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
The file was modifiedllvm/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
The file was modifiedllvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
The file was modifiedllvm/test/CodeGen/ARM/GlobalISel/thumb-select-arithmetic-ops.mir
The file was modifiedllvm/test/CodeGen/ARM/GlobalISel/thumb-select-load-store.mir
The file was modifiedllvm/test/CodeGen/Thumb2/mve-stacksplot.mir
The file was modifiedllvm/test/MC/ARM/invalid-addsub.s
The file was modifiedllvm/test/MC/ARM/register-token-source-loc.s
The file was modifiedllvm/test/CodeGen/Thumb2/peephole-cmp.mir
Commit 5b7612792aeb5b161fdd69997db2a64b08f075b6 by pavel
[lldb/lua] Make convenience_variables.test compatible with lua-5.1
The file was modifiedlldb/test/Shell/ScriptInterpreter/Lua/convenience_variables.test
Commit e65282deca8455d1cc6d83b7016af9aa374f9f89 by pavel
[lldb/DWARF] Don't automatically search dwo unit attributes
This patch removes the code (deep inside DWARFDebugInfoEntry) which
automagically returned the attributes of the dwo unit DIE when asking
for the attributes of the skeleton unit.  This is fairly hacky, and not
consistent with how llvm DWARF parser operates.
Instead, I change the code the explicitly request (via
GetNonSkeletonUnit) the right unit to search (there were just two places
that needed this). If it turns out we need this more often, we can
create a utility function (external to DWARFUnit) for doing this.
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFUnit.h
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFUnit.cpp
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp
Commit 57a51b689e7b99c694a028104b0b5a69b80fd002 by ibiryukov
[CodeComplete] Suggest 'return nullptr' in functions returning pointers
Reviewers: kadircet
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72497
The file was modifiedclang/test/CodeCompletion/patterns.cpp
The file was modifiedclang/lib/Sema/SemaCodeComplete.cpp
Commit c88e298b69409e35e35ab601592197f5a2bc1c30 by pavel
[lldb] Surpress "bitfield too small" gcc warning
Gcc produces this (technically correct) warning when storing an
explicitly-sized enum in a bitfield. Surpress that by changing the type
of the bitfield to an integer. The same approach is used elsewhere in
llvm (e.g. 56b5eab12).
The file was modifiedlldb/include/lldb/Symbol/DebugMacros.h
Commit 564481aebe18a723c9cfe9ea9ca5808771f7e9d8 by andrew.ng
[Support] ThreadPoolExecutor fixes for Windows/MinGW
Changed ThreadPoolExecutor to no longer use detached threads and instead
to join threads on destruction. This is to prevent intermittent crashing
on Windows when doing a normal full exit, e.g. via exit().
Changed ThreadPoolExecutor to be a ManagedStatic so that it can be
stopped on llvm_shutdown(). Without this, it would only be stopped in
the destructor when doing a full exit. This is required to avoid
intermittent crashing on Windows due to a race condition between the
ThreadPoolExecutor starting up threads and the process doing a fast
exit, e.g. via _exit().
The Windows crashes appear to only occur with the MSVC static runtimes
and are more frequent with the debug static runtime.
These changes also prevent intermittent deadlocks on exit with the MinGW
runtime.
Differential Revision: https://reviews.llvm.org/D70447
The file was modifiedllvm/lib/Support/Parallel.cpp
The file was modifiedlld/Common/ErrorHandler.cpp
Commit e4d672971030fe26dbb8237038038c3ff9ae7541 by Tatyana Krasnukha
[lldb][tests] Take into account all parent's categories when traverse
folders upwards
This is needed to not re-write parent's categories by categories of a
nested folder, e.g. commands/expression/completion specify "cmdline"
category, however it still belongs to parent's "expression" category.
The sentinel ".categories" in the test-suite root directory is no longer
needed.
Differential Revision: https://reviews.llvm.org/D71905
The file was modifiedlldb/packages/Python/lldbsuite/test/test_result.py
The file was removedlldb/packages/Python/lldbsuite/test/.categories
Commit 9ba151274869c377921a09ba0bd635412da755ef by Tatyana Krasnukha
[lldb][test] NFC, re-use _getTestPath() function
The file was modifiedlldb/packages/Python/lldbsuite/test/test_result.py
Commit 76e9c2a9870e36415eb343d28942a42296f85597 by ulrich.weigand
[FPEnv] Generate constrained FP comparisons from clang
Update the IRBuilder to generate constrained FP comparisons in
CreateFCmp when IsFPConstrained is true, similar to the other places in
the IRBuilder.
Also, add a new CreateFCmpS to emit signaling FP comparisons, and use it
in clang where comparisons are supposed to be signaling
(currently, only when emitting code for the <, <=, >, >= operators).
Note that there is currently no way to add fast-math flags to a
constrained FP comparison, since this is implemented as an intrinsic
call that returns a boolean type, and FMF are only allowed for calls
returning a floating-point type. However, given the discussion around
https://bugs.llvm.org/show_bug.cgi?id=42179, it seems that FCmp itself
really shouldn't have any FMF either, so this is probably OK.
Reviewed by: craig.topper
Differential Revision: https://reviews.llvm.org/D71467
The file was modifiedllvm/include/llvm/IR/IRBuilder.h
The file was addedclang/test/CodeGen/fpconstrained-cmp-float.c
The file was addedclang/test/CodeGen/fpconstrained-cmp-double.c
The file was modifiedclang/lib/CodeGen/CGExprScalar.cpp
Commit b3af8ab7f83c2a825c584ddedf5cc9207ca66b44 by Tatyana Krasnukha
[lldb][tests] Cleanup '.categories'
The file was removedlldb/packages/Python/lldbsuite/test/commands/command/source/.categories
The file was modifiedlldb/packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-objc/.categories
The file was removedlldb/packages/Python/lldbsuite/test/commands/command/script/.categories
The file was removedlldb/packages/Python/lldbsuite/test/commands/command/history/.categories
The file was removedlldb/packages/Python/lldbsuite/test/commands/command/script_alias/.categories
The file was removedlldb/packages/Python/lldbsuite/test/commands/expression/no-deadlock/.categories
The file was addedlldb/packages/Python/lldbsuite/test/commands/command/.categories
Commit b1bb5ce96d349689085eab38121c85737de1fcaa by diogo.sampaio
Reverting, broke some bots. Need further investigation.
Summary: This reverts commit 8c12769f3046029e2a9b4e48e1645b1a77d28650.
Reviewers:
Subscribers:
The file was modifiedllvm/test/CodeGen/MIR/ARM/thumb2-sub-sp-t3.mir
The file was modifiedllvm/lib/Target/ARM/Thumb2InstrInfo.cpp
The file was modifiedllvm/lib/Target/ARM/ARMBaseInstrInfo.cpp
The file was modifiedllvm/test/MC/ARM/thumb-diagnostics.s
The file was modifiedllvm/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
The file was modifiedllvm/test/CodeGen/Thumb2/peephole-addsub.mir
The file was modifiedllvm/test/MC/ARM/register-token-source-loc.s
The file was modifiedllvm/test/MC/ARM/negative-immediates.s
The file was modifiedllvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb2.txt
The file was removedllvm/test/CodeGen/Thumb2/bug-subw.ll
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb-tests.txt
The file was modifiedllvm/test/CodeGen/Thumb2/peephole-cmp.mir
The file was modifiedllvm/test/CodeGen/ARM/GlobalISel/thumb-select-load-store.mir
The file was modifiedllvm/lib/Target/ARM/ARMInstrThumb2.td
The file was modifiedllvm/test/CodeGen/Thumb2/mve-stacksplot.mir
The file was modifiedllvm/lib/Target/ARM/ARMAsmPrinter.cpp
The file was modifiedllvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
The file was modifiedllvm/test/CodeGen/Thumb2/fp16-stacksplot.mir
The file was modifiedllvm/test/MC/ARM/invalid-addsub.s
The file was modifiedllvm/test/CodeGen/ARM/GlobalISel/thumb-select-arithmetic-ops.mir
The file was modifiedllvm/test/MC/Disassembler/ARM/thumb2-v8.txt
The file was modifiedllvm/test/MC/ARM/basic-thumb2-instructions.s
The file was modifiedllvm/test/MC/Disassembler/ARM/invalid-thumbv7.txt
Commit e49c3c8f2ef97bdf256ca76f3d001eeb79361d56 by benny.kra
Sprinkle some constexpr on default ctors so the compiler can diagnose
unused instances. NFCI.
The file was modifiedmlir/include/mlir/IR/Value.h
The file was modifiedmlir/lib/Dialect/SPIRV/SPIRVOps.cpp
The file was modifiedmlir/include/mlir/IR/Types.h
The file was modifiedmlir/lib/Dialect/VectorOps/VectorOps.cpp
The file was modifiedmlir/include/mlir/IR/Attributes.h
Commit 4569f63ae1cb520ce28f08f4800dfbcd5f255eed by sjoerd.meijer
ARMLowOverheadLoops: a few more dbg msgs to better trace rejected TP
loops. NFC.
The file was modifiedllvm/lib/Target/ARM/ARMLowOverheadLoops.cpp
Commit 2f2f41e12c5201b600d887d22ce5cb4afd2ff594 by jan.kratochvil
RangeDataVector: Support custom sorting for D63540
As suggested by @labath extended RangeDataVector so that user can
provide custom sorting of the Entry's `data' field for D63540.
       https://reviews.llvm.org/D63540
RangeData functions were used just by RangeDataVector (=after I removed
them LLDB still builds fine) which no longer uses them so I removed
them.
Differential revision: https://reviews.llvm.org/D72460
The file was modifiedlldb/unittests/Utility/RangeMapTest.cpp
The file was modifiedlldb/include/lldb/Utility/RangeMap.h
Commit 3772ea9dd9368cfdc73595854c143bc3f16a5ade by sam.parker
[ARM][MVE] Tail predicate VMAX,VMAXA,VMIN,VMINA
Add the MVE min and max instructions to our tail predication whitelist.
Differential Revision: https://reviews.llvm.org/D72502
The file was modifiedllvm/lib/Target/ARM/ARMInstrMVE.td
The file was modifiedllvm/unittests/Target/ARM/MachineInstrTest.cpp