FailedChanges

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

Summary

  1. Remove error in additional '(' that was placed in previous patch. (details)
Commit 53b0602b58d29771a9637826db88c6849212dff0 by 31459023+hctim
Remove error in additional '(' that was placed in previous patch.
The file was modifiedzorg/buildbot/builders/sanitizers/buildbot_cmake.sh

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

Summary

  1. [RISCV] Add riscv{32,64} to ALL_CRT_SUPPORTED_ARCH list (details)
  2. [GWP-ASan] Respect compiler-rt's -fno-lto (details)
  3. [Sema] Suppress -Wchar-subscripts if the index is a literal char (details)
  4. [OPENMP][DOCS] Update OpenMP status (NFC) (details)
  5. [openmp] [test] Skip one more test that kills NetBSD buildbot (details)
  6. comment shiftamountthreshold (details)
  7. [lldb] Comment typo fix (details)
  8. Using crtp to refactor the xcoff section header (details)
  9. AsmWriterOperand - fix uninitialized variable warning. NFCI. (details)
  10. X86CondBrFolding - remove non-existent fixBranchProb function. NFC. (details)
  11. ManagedStringPool - pre-increment iterator. NFC. (details)
  12. PostRAScheduler - fix uninitialized variable warning. NFCI. (details)
  13. TypeRecord - fix uninitialized variable warnings. NFCI. (details)
  14. MachineMemOperand::getBaseAlignment() - fix "shift of i32 then extended (details)
  15. X86FrameLowering - fix bool to unsigned cast static analyzer warnings. (details)
  16. [InstCombine] canonicalize shift+logic+shift to reduce dependency chain (details)
  17. [XCOFF] Move explicit instantions after member function definitions to (details)
  18. [llvm-ar] Support verbose mode for operation 'x' (details)
  19. [XCOFF] Fix link errors from explicit template instantiation (details)
  20. Revert "[XCOFF] Fix link errors from explicit template instantiation" (details)
  21. [XCOFF] Add back extern template declarations (details)
  22. [RISCV] Fix up tests on Windows after new usage of sys::path::append (details)
  23. [AsmWritter] Fixed "null check after dereferencing" warning (details)
  24. Fix warning about unused std::unique result, erase shifted elements (details)
  25. canFoldMergeOpcode returns a bool result not an unsigned. NFCI. (details)
  26. ImutAVLTree::validateTree - fix null dereference typo warning. NFCI. (details)
  27. FDRRecords - fix uninitialized variable warnings. NFCI. (details)
  28. Revert "[Sema] Suppress -Wchar-subscripts if the index is a literal (details)
  29. crashlog.py: Improve regular expressions (details)
  30. Wrong debug info generated at -O2 (-O0 is correct) (details)
Commit dd2588001082d7299f71552029b06fd0620fe23c by ed.jones
[RISCV] Add riscv{32,64} to ALL_CRT_SUPPORTED_ARCH list
This allows crtbegin and crtend to be built, allowing RISC-V to no
longer rely on implementations from libgcc.
Differential revision: https://reviews.llvm.org/D68393
The file was modifiedcompiler-rt/cmake/config-ix.cmake
Commit 343597789eba1e6482e130b0c1b0818b1432d311 by 31459023+hctim
[GWP-ASan] Respect compiler-rt's -fno-lto
https://bugs.llvm.org/show_bug.cgi?id=43722
GWP-ASan didn't include SANITIZER_COMMON_CFLAGS, and thus would produce
LLVM bitcode files, when compiler-rt is generally built without LTO.
The file was modifiedcompiler-rt/lib/gwp_asan/CMakeLists.txt
Commit 7adab7719e55e1b29bfd521dcc73f202139e8f41 by ed.jones
[Sema] Suppress -Wchar-subscripts if the index is a literal char
Assume that the user knows what they're doing if they provide a char
literal as an array index. This more closely matches the behavior of
GCC.
Differential Revision: https://reviews.llvm.org/D58896
The file was modifiedclang/test/SemaCXX/warn-char-subscripts.cpp
The file was modifiedclang/lib/Sema/SemaExpr.cpp
Commit bcf754a3212c12ecc896bac66c599d571eca57d9 by a.bataev
[OPENMP][DOCS] Update OpenMP status (NFC)
Summary: This is updating the OpenMP status table. Cray has volunteered
for `defaultmap` and supporting `in_reduction` on the `target`
construct, so the status on those entries from was changed from
"unclaimed". Also, a new entry was added for supporting non-contiguous
arrays sections on the `target update` directive.
Reviewers: ABataev, hfinkel, jdoerfert, kkwli0
Reviewed By: ABataev
Subscribers: guansong, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69923
The file was modifiedclang/docs/OpenMPSupport.rst
Commit 6f8ee2c5755cb47190bc1aa8c5d8905317e8806f by mgorny
[openmp] [test] Skip one more test that kills NetBSD buildbot
The file was modifiedopenmp/runtime/test/tasking/omp_task_depend_resize_hashmap.c
Commit 0d3d3822f53b12ad292c3a63eb47cbe3ac59d0cf by joan.lluch
comment shiftamountthreshold
The file was modifiedllvm/lib/Target/MSP430/MSP430ISelLowering.cpp
Commit 44f43461c018e7fdc3b1cf3a7a41f52b1cb20a39 by jan.kratochvil
[lldb] Comment typo fix
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/DWARFDeclContext.h
Commit c63c1a72dab96b0d090b2601eb1e7f40df2db534 by diggerlin
Using crtp to refactor the xcoff section header
SUMMARY: According to https://reviews.llvm.org/D68575#inline-617586,
Create a NFC patch for it.
Using crtp to refactor the xcoff section header Move the define of
SectionFlagsReservedMask and SectionFlagsTypeMask from XCOFFDumper.cpp
to XCOFFObjectFile.h
Reviewers: hubert.reinterpretcast,jasonliu Subscribers: rupprecht,
seiyai,hiraditya
Differential Revision: https://reviews.llvm.org/D69131
The file was modifiedllvm/tools/llvm-readobj/XCOFFDumper.cpp
The file was modifiedllvm/include/llvm/Object/XCOFFObjectFile.h
The file was modifiedllvm/lib/Object/XCOFFObjectFile.cpp
Commit bcd7674e06796d409aa85f77a011f701115bff17 by llvm-dev
AsmWriterOperand - fix uninitialized variable warning. NFCI.
The file was modifiedllvm/utils/TableGen/AsmWriterInst.h
Commit f08324067939d299047cf742f7d5224060e2e29c by llvm-dev
X86CondBrFolding - remove non-existent fixBranchProb function. NFC.
The file was modifiedllvm/lib/Target/X86/X86CondBrFolding.cpp
Commit d5c4881a56e0477acb8b38deff7eb3c401427d24 by llvm-dev
ManagedStringPool - pre-increment iterator. NFC.
The file was modifiedllvm/lib/Target/NVPTX/ManagedStringPool.h
Commit 77cfe83f7dd636845915bbb0295013a570778278 by llvm-dev
PostRAScheduler - fix uninitialized variable warning. NFCI.
The file was modifiedllvm/lib/CodeGen/PostRASchedulerList.cpp
Commit 0e9b5760f2c6ce6f41600b90b89a960de41de960 by llvm-dev
TypeRecord - fix uninitialized variable warnings. NFCI.
The file was modifiedllvm/include/llvm/DebugInfo/CodeView/TypeRecord.h
Commit 205c84dc8aee45f29ac92bb7b871ed2845147388 by llvm-dev
MachineMemOperand::getBaseAlignment() - fix "shift of i32 then extended
to i64" static analyzer warning. NFCI.
The file was modifiedllvm/include/llvm/CodeGen/MachineMemOperand.h
Commit 05299c7d98ab5562ebf927847126621826358907 by llvm-dev
X86FrameLowering - fix bool to unsigned cast static analyzer warnings.
NFCI.
The file was modifiedllvm/lib/Target/X86/X86FrameLowering.cpp
Commit d9ccb6367a1089fd61bd85be6b0fbb0d6a590e05 by spatel
[InstCombine] canonicalize shift+logic+shift to reduce dependency chain
shift (logic (shift X, C0), Y), C1 --> logic (shift X, C0+C1), (shift Y,
C1)
This is an IR translation of an existing SDAG transform added here:
rL370617
So we again have 9 possible patterns with a commuted IR variant of each
pattern: https://rise4fun.com/Alive/VlI https://rise4fun.com/Alive/n1m
https://rise4fun.com/Alive/1Vn
Part of the motivation is to allow easier recognition and subsequent
canonicalization of bswap patterns as discussed in PR43146:
https://bugs.llvm.org/show_bug.cgi?id=43146
We had to delay this transform because it used to allow the SLP
vectorizer to create awful reductions out of simple load-combines. That
problem was fixed with: rL375025
(we'll bring back load combining in IR someday...)
The backend is also better equipped to deal with these patterns now
using hooks like TLI.getShiftAmountThreshold().
The only remaining potential controversy is that the -reassociate pass
tends to reverse this kind of pattern (to help GVN?). But since
-reassociate doesn't do anything with these specific patterns, there is
no conflict currently.
Finally, there's a new pass proposal at D67383 for general
tree-height-reduction reassociation, and it could use a cost model to
decide how to optimally rearrange these kinds of ops for a target. That
patch appears to be stalled.
Differential Revision: https://reviews.llvm.org/D69842
The file was modifiedllvm/test/Transforms/InstCombine/bswap.ll
The file was modifiedllvm/test/Transforms/InstCombine/shift-logic.ll
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineShifts.cpp
Commit f8622543ad07f57618c9ecdb3b3a8c7cabe40b85 by maskray
[XCOFF] Move explicit instantions after member function definitions to
fix clang builds
The file was modifiedllvm/lib/Object/XCOFFObjectFile.cpp
Commit 7d2b0ec345487537e37a24b323d612c7c06295cd by maskray
[llvm-ar] Support verbose mode for operation 'x'
Reviewed By: jhenderson, kongyi
Differential Revision: https://reviews.llvm.org/D69911
The file was modifiedllvm/test/tools/llvm-ar/extract.test
The file was modifiedllvm/tools/llvm-ar/llvm-ar.cpp
Commit c989993ba1a666f04f7aee7df51d9f4de0588b71 by rnk
[XCOFF] Fix link errors from explicit template instantiation
I happen to be using clang-cl+lld-link locally, and I get these link
errors:
lld-link: error: undefined symbol: public: unsigned short __cdecl
llvm::object::XCOFFSectionHeader<struct
llvm::object::XCOFFSectionHeader64>::getSectionType(void) const
>>> referenced by
C:\src\llvm-project\llvm\tools\llvm-readobj\XCOFFDumper.cpp:106
>>>             
tools\llvm-readobj\CMakeFiles\llvm-readobj.dir\XCOFFDumper.cpp.obj:(public:
virtual void __cdecl `anonymous
namespace'::XCOFFDumper::printSectionHeaders(void))
I suspect this is because the explicit template instaniation appears
before the inline method definitions in the .cpp file, so they aren't
available at the point of instantiation. Move the explicit instantiation
later.
Also, forward declare the explicit instantiation for good measure.
The file was modifiedllvm/lib/Object/XCOFFObjectFile.cpp
The file was modifiedllvm/include/llvm/Object/XCOFFObjectFile.h
Commit 2cb3bfe9759a134a2e85207643109b69a86c4bf7 by rnk
Revert "[XCOFF] Fix link errors from explicit template instantiation"
This reverts commit c989993ba1a666f04f7aee7df51d9f4de0588b71.
maskray already fixed the explicit instantiation definition in the .cpp
file, and these extern template declarations seem to be causing warnings
that I don't understand.
The file was modifiedllvm/include/llvm/Object/XCOFFObjectFile.h
The file was modifiedllvm/lib/Object/XCOFFObjectFile.cpp
Commit 03495a988da919c975e1635f7e42c8399d71abff by rnk
[XCOFF] Add back extern template declarations
The extern template declarations were fine. The duplicate explicit
instantiations were both in the .cpp file.
The file was modifiedllvm/include/llvm/Object/XCOFFObjectFile.h
Commit f37b5c800e150ad915c4e0571edd2c92c0160d89 by rnk
[RISCV] Fix up tests on Windows after new usage of sys::path::append
The file was modifiedclang/test/Driver/riscv32-toolchain.c
The file was modifiedclang/test/Driver/riscv64-toolchain.c
Commit 6e655e58bc748d631fac90d969e77ceb04b884db by Dávid Bolvanský
[AsmWritter] Fixed "null check after dereferencing" warning
Summary: The 'BB->getParent()' pointer was utilized before it was
verified against nullptr. Check lines: 3567, 3581.
Reviewers: jyknight, RKSimon
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69751
The file was modifiedllvm/lib/IR/AsmWriter.cpp
Commit dd870f6929ee0b1dfb5fd000c7b826a1bd2d2571 by rnk
Fix warning about unused std::unique result, erase shifted elements
This is actually a functional change. I haven't added any new test
coverage. I'm just trying to fix the warning and hoping for the best.
The file was modifiedclang/include/clang/Serialization/ContinuousRangeMap.h
Commit 65c5f4e929e75596732ec49dafe71bc6abea847d by llvm-dev
canFoldMergeOpcode returns a bool result not an unsigned. NFCI.
The file was modifiedllvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h
Commit 4525a43c9393b63e144aaeb8854909b37e3371b0 by llvm-dev
ImutAVLTree::validateTree - fix null dereference typo warning. NFCI.
Noticed by static analyzer.
The file was modifiedllvm/include/llvm/ADT/ImmutableSet.h
Commit 08b5b553083e23008664596a6caa53452a7000c8 by llvm-dev
FDRRecords - fix uninitialized variable warnings. NFCI.
The file was modifiedllvm/include/llvm/XRay/FDRRecords.h
Commit 90ecfa2f5f7feea6e3676d59fe6126e08c52d00e by ed.jones
Revert "[Sema] Suppress -Wchar-subscripts if the index is a literal
char"
This reverts commit 7adab7719e55e1b29bfd521dcc73f202139e8f41.
The file was modifiedclang/lib/Sema/SemaExpr.cpp
The file was modifiedclang/test/SemaCXX/warn-char-subscripts.cpp
Commit ff9d732887385e6f3e516769419dd64b406d81d8 by Adrian Prantl
crashlog.py: Improve regular expressions
This is yet another change to the regular expressions in crashlog.py
that fix a few edge cases, and attempt to improve the readability quite
a bit in the process. My last change to support spaces in filenames
introduced a bug that caused the version/archspec field to be parsed as
part of the image name.
For example, in "0x1111111 - 0x22222 +MyApp Pro arm64 <01234>", the name
of the image was recognized as "MyApp Pro arm64" instead of
"MyApp Pro" with a "version" of arm64.
The bugfix makes the space following an optional field mandatory
*inside* the optional group.
rdar://problem/56883435
Differential Revision: https://reviews.llvm.org/D69871
The file was modifiedlldb/examples/python/crashlog.py
The file was modifiedlldb/test/Shell/Python/crashlog.test
Commit a087b78bc4c31be5a82eda62f1e4c7fe1fec14ad by Vedant Kumar
Wrong debug info generated at -O2 (-O0 is correct)
Instcombiner pass was erasing trivially dead instruction without
updating dependent llvm.dbg.value. which was not showing programmer
current state of variables while debugging. As a part of this fix I did
following, Iterate throught all the users (llvm.dbg) of a instruction
which is trivially dead and set each if them undef, Before deleting the
instruction. Now user will see optimized out, when try to print those
variables. This fixes https://bugs.llvm.org/show_bug.cgi?id=43893
This is my first fix to llvm.
Patch by kamlesh kumar!
Differential Revision: https://reviews.llvm.org/D69809
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineInternal.h
The file was modifiedllvm/lib/Transforms/Utils/Local.cpp
The file was modifiedllvm/test/Transforms/InstCombine/debuginfo-dce.ll
The file was modifiedllvm/test/Transforms/InstCombine/stacksave-debuginfo.ll
The file was modifiedllvm/test/Transforms/InstCombine/cast-mul-select.ll
The file was addedllvm/test/Transforms/InstCombine/pr43893.ll
The file was modifiedllvm/include/llvm/Transforms/Utils/Local.h
The file was modifiedllvm/lib/Transforms/InstCombine/InstructionCombining.cpp