FailedChanges

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

Summary

  1. [FIX] Order macros after D72304 (details)
  2. [NFC] Revert unnecessary parts of b91c267380 (details)
  3. [Attributor] Collect memory accesses with their respective kind and location (details)
  4. [Debuginfo][NFC] Create common error handlers for DWARFContext. (details)
  5. Fix standalone build interaction with compiler extension (details)
  6. [cxx_status] New papers from Prague 2020 WG21 meeting. (details)
  7. PR44890: Inherit explicitly-specified template arguments into base class (details)
  8. [Debuginfo][NFC] Remove usages of WithColor::error and WithColor::warning. (details)
  9. [gn build] unbreak win build by removing flags that only work with a sysroot (details)
  10. [AArch64] Fixup kill flags on BSL generation (details)
  11. [APInt] Add some procedural APInt::byteSwap unit tests (details)
  12. [APInt] byteSwap - simplify sub 64-bits cases to match general implementation. NFCI. (details)
  13. [AArch64][FIX] Correct register live range during pseudo expansion. (details)
  14. [APInt] byteSwap - handle any whole byte bitwidth greater than 16-bits (details)
  15. Fix boolean/bitwise operator precedence warnings. NFCI. (details)
  16. [Support] In tests, fix warning: variable ‘Threads’ set but not used (details)
  17. [obj2yaml] - Fix a -Wsign-compare warning gived by GCC 9.2 (details)
  18. Fix gcc9.2 -Winit-list-lifetime warning. NFCI. (details)
  19. [MC] De-capitalize MCStreamer::Emit{Bundle,Addrsig}* etc (details)
  20. Recommit "[SCCP] Remove forcedconstant, go to overdefined instead" (details)
  21. [X86] Move combineIncDecVector logic from Select to PreprocessISelDAG. (details)
  22. [X86] Add test showing failure to combine shuffle to bit rotation (details)
Commit a0236de7a927d2b7d68a805c40cdf6d4aa9442ba by johannes
[FIX] Order macros after D72304
The file was modifiedllvm/include/llvm/Frontend/OpenMP/OMPKinds.def
Commit a8f039632d6f4d78a1304028df027cd4d0176535 by johannes
[NFC] Revert unnecessary parts of b91c267380

In b91c267380 I accidentally introduced fixes that were not necessary
after 1a93285c686a. All but the `llvm_unreachable` are reverted again.
The file was modifiedllvm/lib/Frontend/OpenMP/OMPContext.cpp
Commit ef746aa11f7697d1128a5900ab40a6776fb5ba0f by johannes
[Attributor] Collect memory accesses with their respective kind and location

In addition to a single bit per memory locations, e.g., globals and
arguments, we now collect more information about the actual accesses,
e.g., what instruction caused it, was it a read/write/read+write, and
what the underlying base pointer was. Follow up patches will make
explicit use of this.

Reviewed By: uenoku

Differential Revision: https://reviews.llvm.org/D73527
The file was modifiedllvm/include/llvm/Transforms/IPO/Attributor.h
The file was modifiedllvm/test/Transforms/Attributor/noreturn_sync.ll
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
The file was modifiedllvm/test/Transforms/Attributor/nofree.ll
The file was modifiedllvm/test/Transforms/Attributor/noreturn_async.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/attrs.ll
The file was modifiedllvm/test/Transforms/Attributor/liveness.ll
The file was modifiedllvm/test/Transforms/Attributor/memory_locations.ll
Commit c187364d406182d8970032b5ed2b2823f9500e97 by a.v.lapshin
[Debuginfo][NFC] Create common error handlers for DWARFContext.

Summary:
this review is extracted from D74308.

It creates two error handlers which allow to redefine error
reporting routine and should be used for all places
where errors are reported:

  std::function<void(Error)> RecoverableErrorHandler = defaultErrorHandler;
  std::function<void(Error)> WarningHandler = defaultWarningHandler;

It also creates accessors to above handlers which should be used to
report errors.

  function_ref<void(Error)> getRecoverableErrorHandler() {
    return RecoverableErrorHandler;
  }

  function_ref<void(Error)> getWarningHandler() { return WarningHandler; }

It patches all error reporting places inside DWARFContext and DWARLinker.

Reviewers: jhenderson, dblaikie, probinson, aprantl, JDevlieghere

Reviewed By: jhenderson, JDevlieghere

Subscribers: hiraditya, llvm-commits

Tags: #llvm, #debug-info

Differential Revision: https://reviews.llvm.org/D74481
The file was modifiedllvm/lib/DebugInfo/DWARF/DWARFContext.cpp
The file was modifiedllvm/include/llvm/DebugInfo/DIContext.h
The file was modifiedllvm/include/llvm/DebugInfo/DWARF/DWARFContext.h
The file was modifiedllvm/lib/Support/WithColor.cpp
The file was modifiedllvm/lib/DWARFLinker/DWARFLinker.cpp
The file was modifiedllvm/include/llvm/Support/WithColor.h
Commit 87dac7da68ea1e0adac78c59ef1891dcf9632b67 by sguelton
Fix standalone build interaction with compiler extension

As suggested in https://github.com/llvm/llvm-project/issues/120, don't try to
generate the extension file from clang, only do the linking step.

Fixes the regression introduced in D74464 when running cmake inside the clang
directory.

Differential Revision: https://reviews.llvm.org/D74602
The file was modifiedllvm/cmake/modules/AddLLVM.cmake
The file was modifiedclang/CMakeLists.txt
Commit b74a381296eef048911bb22dc4eb2d3598460470 by richard
[cxx_status] New papers from Prague 2020 WG21 meeting.
The file was modifiedclang/www/cxx_status.html
Commit 34bd51f4b1d9f489e61becb662bdc72bb56dd277 by richard
PR44890: Inherit explicitly-specified template arguments into base class
deduction.
The file was modifiedclang/lib/Sema/SemaTemplateDeduction.cpp
The file was modifiedclang/test/SemaTemplate/deduction.cpp
The file was modifiedclang/include/clang/Sema/TemplateDeduction.h
Commit 98e3f19b419f3efde3aadca79c3ead75cbbf1b9a by a.v.lapshin
[Debuginfo][NFC] Remove usages of WithColor::error and WithColor::warning.

Summary:
This patch is extracted from D74308.

It patches all usages of WithColor::error() and WithColor::warning
in DebugInfoDWARF library.

Depends on D74481

Reviewers: jhenderson, dblaikie, probinson, aprantl, JDevlieghere

Reviewed By: JDevlieghere

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74635
The file was modifiedllvm/lib/DebugInfo/DWARF/DWARFDebugAranges.cpp
The file was modifiedllvm/include/llvm/DebugInfo/DWARF/DWARFDebugAranges.h
The file was modifiedllvm/lib/DebugInfo/DWARF/DWARFUnit.cpp
The file was modifiedllvm/lib/DebugInfo/DWARF/DWARFDie.cpp
Commit 37d11205724bb5b4541b06bdafe1e11b7ae3614d by thakis
[gn build] unbreak win build by removing flags that only work with a sysroot
The file was modifiedllvm/utils/gn/build/BUILD.gn
Commit da147ef0a5c6d31c21d31a52b97235a629830c15 by david.green
[AArch64] Fixup kill flags on BSL generation

This hopefully fixes up the expensive checks bot.
The file was modifiedllvm/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp
The file was modifiedllvm/test/CodeGen/AArch64/arm64-promote-const.ll
Commit 4d5c3ade0b26ae3189e4601ce539844f54722ed8 by llvm-dev
[APInt] Add some procedural APInt::byteSwap unit tests

rGf0181cc7bac3 added specific tests up to i64, this adds a general loop to test some basic byte moves for larger APInts.
The file was modifiedllvm/unittests/ADT/APIntTest.cpp
Commit 34ad4cca7245626c3d8348c7e3dd077ff35db84e by llvm-dev
[APInt] byteSwap - simplify sub 64-bits cases to match general implementation. NFCI.

We can just byteSwap the entire uint64_t VAL and then shift down into place like we do for the multi-word case.
The file was modifiedllvm/lib/Support/APInt.cpp
Commit dc0b8159890134a59fdf34b20e8b2052d9456441 by Pavel.Iliin
[AArch64][FIX] Correct register live range during pseudo expansion.

This commit fixes the broken tests after
commit b6a9fe209992789be3ed95664d25196361cfad34
on the expensive check builder:
http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-ubuntu/builds/2884
The file was modifiedllvm/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp
Commit 366ae9c90fe6d3a590c0a0d4283c019f11799516 by llvm-dev
[APInt] byteSwap - handle any whole byte bitwidth greater than 16-bits

As noted on D74621, the bswap intrinsic has a self imposed limitation that the type's bitwidth must be divisible by 16, but there's no reason that APInt::byteSwap must have the same limitation, given that it can already handle any byte width.
The file was modifiedllvm/lib/Support/APInt.cpp
The file was modifiedllvm/unittests/ADT/APIntTest.cpp
Commit 8a48c4a97c416825835496fed3c2482b42e52c61 by llvm-dev
Fix boolean/bitwise operator precedence warnings. NFCI.
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
Commit b25fc4123c77097c05ea221e023fa5c6a16e0f41 by alexandre.ganea
[Support] In tests, fix warning: variable ‘Threads’ set but not used
The file was modifiedllvm/unittests/Support/ThreadPool.cpp
Commit b7c8a8e834afbbe9da9cec185d212a26ffd7aefb by grimar
[obj2yaml] - Fix a -Wsign-compare warning gived by GCC 9.2

I was reported that with commit:
https://github.com/llvm/llvm-project/commit/d3963051c490

gcc-9.2 is giving the warning below.
This should help (I have no gcc 9.2 to test).

[ 57%] Building CXX object tools/obj2yaml/CMakeFiles/obj2yaml.dir/elf2yaml.cpp.o
/llvm/tools/obj2yaml/elf2yaml.cpp: In instantiation of ‘llvm::Expected<llvm::ELFYAML::Object*>
{anonymous}::ELFDumper<ELFT>::dump() [with ELFT = llvm::object::ELFType<llvm::support::little, false>]’:
/llvm/tools/obj2yaml/elf2yaml.cpp:1218:31:   required from ‘llvm::Error elf2yaml(llvm::raw_ostream&,
const llvm::object::ELFFile<ELFT>&) [with ELFT = llvm::object::ELFType<llvm::support::little, false>]’
/llvm/tools/obj2yaml/elf2yaml.cpp:1231:47:   required from here
/llvm/tools/obj2yaml/elf2yaml.cpp:207:41: warning: comparison of integer expressions of different
signedness: ‘llvm::support::detail::packed_endian_specific_integral<unsigned int, llvm::support::little, 1>::value_type’ {aka ‘unsigned int’} and ‘int’ [-Wsign-compare]
  207 |     if (!SymTab || SymTabShndx->sh_link != SymTab - Sections.begin())
/llvm/tools/obj2yaml/elf2yaml.cpp: In instantiation of ‘llvm::Expected<llvm::ELFYAML::Object*>
{anonymous}::ELFDumper<ELFT>::dump() [with ELFT = llvm::object::ELFType<llvm::support::big, false>]’:
...
The file was modifiedllvm/tools/obj2yaml/elf2yaml.cpp
Commit ce2b5f1569ad2fff7e5a49d99066b29fd9338abb by llvm-dev
Fix gcc9.2 -Winit-list-lifetime warning. NFCI.

Reported by @lbenes (Luke Benes)
The file was modifiedllvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
Commit 549b436beb4129854e729a3e1398f03429149691 by maskray
[MC] De-capitalize MCStreamer::Emit{Bundle,Addrsig}* etc

So far, all non-COFF-related Emit* functions have been de-capitalized.
The file was modifiedllvm/lib/MC/MCWinCOFFStreamer.cpp
The file was modifiedllvm/lib/Target/XCore/XCoreAsmPrinter.cpp
The file was modifiedllvm/include/llvm/MC/MCXCOFFStreamer.h
The file was modifiedllvm/lib/MC/MCAsmStreamer.cpp
The file was modifiedllvm/lib/Target/Mips/MipsAsmPrinter.cpp
The file was modifiedllvm/lib/MC/MCParser/ELFAsmParser.cpp
The file was modifiedllvm/lib/MC/MCWasmStreamer.cpp
The file was modifiedllvm/lib/Target/Mips/AsmParser/MipsAsmParser.cpp
The file was modifiedllvm/lib/MC/MCXCOFFStreamer.cpp
The file was modifiedllvm/lib/Target/ARM/ARMAsmPrinter.cpp
The file was modifiedllvm/lib/MC/MCParser/WasmAsmParser.cpp
The file was modifiedllvm/lib/Target/AArch64/AArch64AsmPrinter.cpp
The file was modifiedllvm/lib/MC/MCNullStreamer.cpp
The file was modifiedllvm/lib/MC/MCELFStreamer.cpp
The file was modifiedllvm/lib/MC/MCStreamer.cpp
The file was modifiedllvm/lib/Target/Mips/MCTargetDesc/MipsNaClELFStreamer.cpp
The file was modifiedllvm/lib/MC/MCObjectStreamer.cpp
The file was modifiedllvm/include/llvm/MC/MCObjectStreamer.h
The file was modifiedllvm/lib/CodeGen/AsmPrinter/AsmPrinterInlineAsm.cpp
The file was modifiedllvm/include/llvm/MC/MCELFStreamer.h
The file was modifiedllvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
The file was modifiedllvm/lib/Target/X86/X86AsmPrinter.cpp
The file was modifiedllvm/lib/MC/MCParser/DarwinAsmParser.cpp
The file was modifiedllvm/lib/MC/MCParser/AsmParser.cpp
The file was modifiedllvm/lib/Target/NVPTX/MCTargetDesc/NVPTXTargetStreamer.cpp
The file was modifiedllvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
The file was modifiedllvm/include/llvm/MC/MCStreamer.h
The file was modifiedllvm/lib/MC/MCMachOStreamer.cpp
The file was modifiedclang/tools/driver/cc1as_main.cpp
The file was modifiedllvm/lib/Target/Hexagon/MCTargetDesc/HexagonMCELFStreamer.cpp
The file was modifiedllvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
The file was modifiedllvm/include/llvm/MC/MCWinCOFFStreamer.h
The file was modifiedllvm/include/llvm/MC/MCWasmStreamer.h
Commit f8045b250d80ccdb2ae5f46d3aa02137fe475907 by flo
Recommit "[SCCP] Remove forcedconstant, go to overdefined instead"

This includes a fix for cases where things get marked as overdefined in
ResolvedUndefsIn, but we later discover a constant. To avoid crashing,
we consistently bail out on overdefined values in the visitors. This is
similar to the previous behavior with forcedconstant.

This reverts the revert commit 02b72f564c8be0b4f4337d5c4a3fcf7e8018a818.
The file was modifiedllvm/test/Transforms/IPConstantProp/PR16052.ll
The file was modifiedllvm/test/Transforms/SCCP/2006-12-19-UndefBug.ll
The file was modifiedllvm/test/Transforms/SCCP/logical-nuke.ll
The file was modifiedllvm/test/Transforms/IPConstantProp/PR26044.ll
The file was modifiedllvm/test/Transforms/SCCP/switch-multiple-undef.ll
The file was modifiedllvm/test/Transforms/SCCP/ub-shift.ll
The file was modifiedllvm/test/Transforms/SCCP/apint-bigint2.ll
The file was modifiedllvm/test/Transforms/SCCP/undef-resolve.ll
The file was modifiedllvm/test/Transforms/SCCP/apint-select.ll
The file was modifiedllvm/test/Transforms/SCCP/apint-ipsccp3.ll
The file was modifiedllvm/lib/Transforms/Scalar/SCCP.cpp
The file was addedllvm/test/Transforms/SCCP/resolvedundefsin-tracked-fn.ll
The file was modifiedllvm/test/Transforms/SCCP/ip-constant-ranges.ll
The file was modifiedllvm/test/Transforms/SCCP/ipsccp-basic.ll
Commit 3f7649799bedfaa97ced81c95414af23174f811e by craig.topper
[X86] Move combineIncDecVector logic from Select to PreprocessISelDAG.

This allows it to work properly with masked inc/dec for avx512. Those
would have a vselect as the root node so didn't get a chance to call
combineIncDecVector.

This also simplifies the logic because we don't have to manage
the topological ordering.
The file was modifiedllvm/lib/Target/X86/X86ISelDAGToDAG.cpp
The file was modifiedllvm/test/CodeGen/X86/avx512-arith.ll
Commit 4abbaceea07f74c7ed0b2a5b7b452d688e80d24a by llvm-dev
[X86] Add test showing failure to combine shuffle to bit rotation
The file was modifiedllvm/test/CodeGen/X86/vector-shuffle-combining-xop.ll