FailedChanges

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

Summary

  1. [ms] Fix Microsoft compatibility handling of commas in nested macro (details)
  2. [lit] Better/earlier errors when no tests are executed (details)
  3. [lit] Move measurement of testing time out of Run.execute (details)
  4. [lldb] [Process/NetBSD] Add register info for missing register sets (details)
  5. [DAGCombine][MSP430] use shift amount threshold in DAGCombine (2/2) (details)
  6. [SimplifyCFG] Use a (trivially) dominanting widenable branch to remove (details)
  7. [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc (details)
  8. clang/Modules: Bring back optimization lost in 31e14f41a21f (details)
  9. Fix warning: format specifies type 'unsigned long' but the argument has (details)
  10. Remove unused variables, as suggested by @mcgov. (details)
  11. [demangle] NFC: get rid of NodeOrString (details)
  12. [AMDGPU] deduplicate tablegen predicates (details)
  13. [LLDB][Python] remove ArgInfo::count (details)
  14. [OPENMP50]Support for imperfectly nested loops. (details)
  15. Add release notes for commit ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75. (details)
  16. [OPENMP][DOCS]Update list of implemented features, NFC. (details)
  17. [AMDGPU] Added assert in SIFoldOperands before ptr use. NFC. (details)
  18. [AST][NFC] Fixes a comment typo (details)
Commit be6ac471f613427f3b5b3a306fe033e526d59f76 by epastor
[ms] Fix Microsoft compatibility handling of commas in nested macro
expansions.
In Microsoft-compatibility mode, single commas from nested macro
expansions should not be considered as argument separators; we already
emulated this by marking them to be ignored. However, in MSVC's
preprocessor, subsequent expansions DO treat these commas as argument
separators... so we now ignore each comma at most once.
Includes a small unit test that validates we match MSVC's behavior as
shown in https://gcc.godbolt.org/z/y0twaq
Fixes PR43282
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69626
The file was modifiedclang/lib/Lex/PPMacroExpansion.cpp
The file was modifiedclang/test/Preprocessor/microsoft-ext.c
Commit d8f2bff75126c6dde694ad245f9807fa12ad5630 by julian.lettner
[lit] Better/earlier errors when no tests are executed
Fail early, when we discover no tests at all, or filter out all of them.
The file was modifiedllvm/utils/lit/lit/main.py
The file was modifiedllvm/utils/lit/lit/cl_arguments.py
The file was modifiedllvm/utils/lit/lit/run.py
The file was modifiedllvm/utils/lit/tests/selecting.py
Commit bd14bb42f03a227c7a83db942af4680d2fe0a78d by julian.lettner
[lit] Move measurement of testing time out of Run.execute
The file was modifiedllvm/utils/lit/lit/main.py
The file was modifiedllvm/utils/lit/lit/run.py
Commit 6eca4f46912a8318d7a5888506c3f26c20bdc012 by mgorny
[lldb] [Process/NetBSD] Add register info for missing register sets
Add info for all register sets supported in NetBSD, particularly for all
registers 'expected' by LLDB.  This is necessary in order to fix
python_api/lldbutil/iter/TestRegistersIterator.py test that currently
fails due to missing names of register sets (None).
This copies fpreg descriptions from Linux, and combines Linux' AVX and
MPX registers into a single XState group, to fit NetBSD register group
design.  Technically, we do not support MPX registers at the moment but
gdb-remote insists on passing their errors anyway, and if we do not
include it in any group, they end up in a separate anonymous group that
breaks the test.
While at it, swap the enums for XState and DBRegs to match register set
ordering.
This also adds a few consts to the lldb-x86-register-enums.h to provide
more consistency between user registers and debug registers.
Differential Revision: https://reviews.llvm.org/D69667
The file was modifiedlldb/source/Plugins/Process/NetBSD/NativeRegisterContextNetBSD_x86_64.cpp
The file was modifiedlldb/packages/Python/lldbsuite/test/commands/register/register/register_command/TestRegisters.py
The file was modifiedlldb/source/Plugins/Process/NetBSD/NativeRegisterContextNetBSD_x86_64.h
The file was modifiedlldb/source/Plugins/Process/Utility/lldb-x86-register-enums.h
Commit 113181e9bd05353ed562ee7b971bf7f1e58cd5de by spatel
[DAGCombine][MSP430] use shift amount threshold in DAGCombine (2/2)
Continuation of: D69116
Contributes to a fix for PR43559:
https://bugs.llvm.org/show_bug.cgi?id=43559
See also D69099 and D69116
Use the TLI hook in DAGCombine.cpp to guard against creating shift nodes
that are not optimal for a target.
Patch by: @joanlluch (Joan LLuch)
Differential Revision: https://reviews.llvm.org/D69120
The file was modifiedllvm/test/CodeGen/MSP430/shift-amount-threshold.ll
The file was modifiedllvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Commit 6ff439b57f0fc2b1a2193ae37637c531ff652b1c by listmail
[SimplifyCFG] Use a (trivially) dominanting widenable branch to remove
later slow path blocks
This transformation is a variation on the GuardWidening transformation
we have checked in as it's own pass. Instead of focusing on merge (i.e.
hoisting and simplifying) two widenable branches, this transform makes
the observation that simply removing a second slowpath block (by reusing
an existing one) is often a very useful canonicalization. This may lead
to later merging, or may not. This is a useful generalization when the
intermediate block has loads whose dereferenceability is hard to
establish.
As noted in the patch, this can be generalized further, and will be.
Differential Revision: https://reviews.llvm.org/D69689
The file was addedllvm/test/Transforms/SimplifyCFG/wc-widen-block.ll
The file was modifiedllvm/lib/Transforms/Utils/SimplifyCFG.cpp
Commit b2b6a54f847f33f821f41e3e82bf3b86e08817a0 by craig.topper
[X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc
-mvzeroupper will force the vzeroupper insertion pass to run on CPUs
that normally wouldn't. -mno-vzeroupper disables it on CPUs where it
normally runs.
To support this with the default feature handling in clang, we need a
vzeroupper feature flag in X86.td. Since this flag has the opposite
polarity of the fast-partial-ymm-or-zmm-write we used to use to disable
the pass, we now need to add this new flag to every CPU except KNL/KNM
and BTVER2 to keep identical behavior.
Remove -fast-partial-ymm-or-zmm-write which is no longer used.
Differential Revision: https://reviews.llvm.org/D69786
The file was modifiedllvm/lib/Target/X86/X86TargetTransformInfo.h
The file was modifiedllvm/lib/Target/X86/X86.td
The file was modifiedllvm/docs/ReleaseNotes.rst
The file was modifiedllvm/test/CodeGen/X86/avx-vzeroupper.ll
The file was modifiedclang/docs/ReleaseNotes.rst
The file was modifiedclang/include/clang/Driver/Options.td
The file was modifiedllvm/lib/Target/X86/X86VZeroUpper.cpp
The file was modifiedclang/test/Driver/x86-target-features.c
The file was modifiedllvm/lib/Target/X86/X86Subtarget.h
Commit 8112a423a8ede9bce64b6553e6451bf10995105c by Duncan P. N. Exon Smith
clang/Modules: Bring back optimization lost in 31e14f41a21f
31e14f41a21f9016050a20f07d5da03db2e8c13e accidentally dropped caching of
failed module loads.  This brings it back by making
ModuleMap::getCachedModuleLoad return an Optional.
The file was modifiedclang/lib/Frontend/CompilerInstance.cpp
The file was modifiedclang/include/clang/Lex/ModuleMap.h
Commit 9cc3ebf8b7630e30bc0eea6dc4a55348edf71091 by alexandre.ganea
Fix warning: format specifies type 'unsigned long' but the argument has
type 'unsigned long long' [-Wformat]
The file was modifiedcompiler-rt/lib/sanitizer_common/tests/sanitizer_bitvector_test.cpp
Commit efad56b2be9b9f7d5b62d1f06541192fa6b537ee by alexandre.ganea
Remove unused variables, as suggested by @mcgov.
Fixes warning: unused variable 'XXX' [-Wunused-const-variable]
The file was modifiedcompiler-rt/lib/asan/asan_malloc_win.cpp
Commit af11f417fc7d2390da4a883c05c098f23891862e by erik.pilkington
[demangle] NFC: get rid of NodeOrString
This class was a bit overengineered, and was triggering some PVS
warnings. Instead, put strings into a NameType and let clients
unconditionally treat it as a Node.
The file was modifiedllvm/include/llvm/Demangle/ItaniumDemangle.h
The file was modifiedllvm/lib/Demangle/ItaniumDemangle.cpp
The file was modifiedlibcxxabi/src/cxa_demangle.cpp
The file was modifiedlibcxxabi/src/demangle/ItaniumDemangle.h
The file was modifiedllvm/lib/Support/ItaniumManglingCanonicalizer.cpp
Commit 4312c4afd43209400df53ca541b4b19919f797af by Stanislav.Mekhanoshin
[AMDGPU] deduplicate tablegen predicates
We are duplicating predicates if several parts of the combined predicate
list contain the same condition. Added code to deduplicate the list.
We have AssemblerPredicates and AssemblerPredicate in the
PredicateControl, but we never use AssemblerPredicates with an actual
list, so this one is dropped.
This addresses the first part of the llvm bug 43886:
https://bugs.llvm.org/show_bug.cgi?id=43886
Differential Revision: https://reviews.llvm.org/D69815
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.td
The file was modifiedllvm/lib/Target/AMDGPU/VOP3Instructions.td
The file was modifiedllvm/lib/Target/AMDGPU/DSInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/MIMGInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/VOP3PInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/SOPInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/VOPCInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/SMInstructions.td
The file was modifiedllvm/lib/Target/AMDGPU/VOP1Instructions.td
The file was modifiedllvm/lib/Target/AMDGPU/VOP2Instructions.td
Commit adbf64ccc9e18278600ebaeadd8f0117eb8e64b1 by lawrence_danna
[LLDB][Python] remove ArgInfo::count
Summary: This patch updates the last user of ArgInfo::count and deletes
it.   I also delete `GetNumInitArguments()` and `GetInitArgInfo()`.
Classess are callables and `GetArgInfo()` should work on them.
On python 3 it already works, of course. `inspect` is good.
On python 2 we have to add yet another special case.   But hey if python
2 wasn't crufty we wouln't need python 3.
I also delete `is_bound_method` becuase it is unused.
This path is tested in `TestStepScripted.py`
Reviewers: labath, mgorny, JDevlieghere
Reviewed By: labath, JDevlieghere
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D69742
The file was modifiedlldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h
The file was modifiedlldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp
The file was modifiedlldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
The file was modifiedlldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp
The file was modifiedlldb/scripts/Python/python-wrapper.swig
Commit 8bbf2e37167d9ac08fa9d3c772d48ca7d9a6f8f6 by a.bataev
[OPENMP50]Support for imperfectly nested loops.
Added support for imperfectly nested loops introduced in OpenMP 5.0.
The file was modifiedclang/lib/AST/StmtOpenMP.cpp
The file was modifiedclang/lib/CodeGen/CGStmtOpenMP.cpp
The file was modifiedclang/lib/Sema/SemaOpenMP.cpp
The file was modifiedclang/include/clang/AST/StmtOpenMP.h
The file was modifiedclang/test/OpenMP/for_ast_print.cpp
The file was modifiedclang/test/OpenMP/for_codegen.cpp
The file was modifiedclang/test/OpenMP/for_collapse_messages.cpp
Commit d11a9018b773c0359934a7989d886b02468112e4 by jyknight
Add release notes for commit ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75.
(Which was "[ObjC] Diagnose implicit type coercion from ObjC 'Class' to
object pointer types.")
The file was modifiedclang/docs/ReleaseNotes.rst
Commit 3eecd601ed803ed3dc15b054cdddee68702ba3dd by a.bataev
[OPENMP][DOCS]Update list of implemented features, NFC.
The file was modifiedclang/docs/OpenMPSupport.rst
Commit 1bfcc60828d6e1fb93d65b53986514c059b1263d by Stanislav.Mekhanoshin
[AMDGPU] Added assert in SIFoldOperands before ptr use. NFC.
The file was modifiedllvm/lib/Target/AMDGPU/SIFoldOperands.cpp
Commit 403739b2fdb64e90118017355bd01f88a0640b3f by koraq
[AST][NFC] Fixes a comment typo
Also a test for commit access.
The file was modifiedclang/lib/AST/DeclCXX.cpp