FailedChanges

Summary

  1. Follow-up build fix for rGae6f78824031 (details)
  2. [AArch64] add tests for store merge of truncs; NFC (details)
  3. [libunwind] Ensure enough alignment for unw_cursor_t for SEH build configurations (details)
  4. [flang][msvc] Split class declaration and constexpr variable definition. NFC. (details)
  5. [LLD][COFF] Reset outputSections for successive runs (details)
  6. Revert "[Polly] Ensure consistent Scop::InstStmtMap. NFC." (details)
Commit 93af37043b9c1a8ea47890ad2bcf277f9b36c47d by jeremy.morse
Follow-up build fix for rGae6f78824031

One of the bots objects to brace-initializing a tuple:

  http://lab.llvm.org:8011/builders/clang-cmake-x86_64-sde-avx512-linux/builds/43595/steps/build%20stage%201/logs/stdio

As the tuple constructor is apparently explicit. Fall back to the (not
as pretty) explicit construction of a tuple. I'd thought this was
permitted behaviour; will investigate why this fails later.
The file was modifiedllvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
Commit c1dc44f914e250979cf95dd486a3b87d2d692bd4 by spatel
[AArch64] add tests for store merge of truncs; NFC
The file was addedllvm/test/CodeGen/AArch64/merge-trunc-store.ll
Commit 542db87f1ac67087aeb5cf572b1f9744d482da69 by martin
[libunwind] Ensure enough alignment for unw_cursor_t for SEH build configurations

When built in SEH mode, UnwindCursor contains a CONTEXT struct,
which is aligned to 16 bytes in most configurations, causing the
whole UnwindCursor object to have 16 byte alignment.

This fixes backtraces using _Unwind_Backtrace on x86_64 mingw,
where an unw_cursor_t allocated on the stack was misaligned before.

This is an ABI break for this struct for this configuration, but very
few callers call libunwind directly (and even fewer directly allocate
an unw_cursor_t anyway).

Differential Revision: https://reviews.llvm.org/D86102
The file was modifiedlibunwind/include/libunwind.h
The file was modifiedlibunwind/src/UnwindCursor.hpp
Commit 207d4499dfab050b0535c7361527d3c37cf6488a by llvm-project
[flang][msvc] Split class declaration and constexpr variable definition. NFC.

Msvc has trouble defining a struct/class and defining a constexpr symbol in the same declarator. It reports the following error:
```
basic-parsers.h(809): error C2131: expression did not evaluate to a constant
basic-parsers.h(809): note: failure was caused by call of undefined function or one not declared 'constexpr'
basic-parsers.h(809): note: see usage of 'Fortran::parser::OkParser::OkParser'
```

Fix the msvc compilation by splitting the two definitions into two separate declarators.

This patch is part of the series to [[ http://lists.llvm.org/pipermail/flang-dev/2020-July/000448.html | make flang compilable with MS Visual Studio ]].

Reviewed By: DavidTruby, klausler

Differential Revision: https://reviews.llvm.org/D85937
The file was modifiedflang/lib/Parser/basic-parsers.h
The file was modifiedflang/lib/Parser/expr-parsers.cpp
The file was modifiedflang/lib/Parser/token-parsers.h
Commit 54f5a4ea4c859cf7f34f0d4955abc3a2f44bd0dc by martin
[LLD][COFF] Reset outputSections for successive runs

The global variable outputSections in the COFF writer was not
cleared between runs which caused successive calls to lld::coff::link
to generate invalid binaries. These binaries when loaded would result
in "invalid win32 applications" and/or "bad image" errors.

Differential Revision: https://reviews.llvm.org/D86401
The file was modifiedlld/COFF/Writer.cpp
Commit 8b14db636dec8afa904366a0d5499b80d7dc7034 by llvm-project
Revert "[Polly] Ensure consistent Scop::InstStmtMap. NFC."

This reverts commit 8e06bf6b3a2e8d25e56cd52dca0cf3ff1b37b5d1.

It broke the polly-x86_64-linux-test-suite and
aosp-O3-polly-before-vectorizer-unprofitable buildbots.
The file was modifiedpolly/lib/Analysis/ScopInfo.cpp
The file was modifiedpolly/include/polly/ScopInfo.h