SuccessChanges

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

Summary

  1. ContentCache: Simplify by always owning the MemoryBuffer (details)
  2. [flang] Fix call to CHECK() on erroneous subroutine declaration (details)
  3. [lldb] Don't strip LLDB.framework on install (details)
  4. SourceManager: Simplify early returns in ContentCache::getBufferOrNone, NFC (details)
  5. [llvm] Fix ODRViolations for VersionTuple YAML specializations NFC (details)
  6. [test] Fix -fbasic-block-sections= test on Windows after D89500 (details)
  7. This is a test commit (details)
  8. Revert "This is a test commit" (details)
  9. [Polly] Delete unused lambda capture after 7175cffb2133048018df74c1b49d1d4962ea18f2 (details)
Commit 296314516d103f8eeb987a08b509c1381cfeef89 by Duncan P. N. Exon Smith
ContentCache: Simplify by always owning the MemoryBuffer

This changes `ContentCache::Buffer` to use
`std::unique_ptr<MemoryBuffer>` instead of the `PointerIntPair`. It
drops the (mostly unused) `DoNotFree` bit, instead creating a (new)
non-owning `MemoryBuffer` instance when passed a `MemoryBufferRef`.

Differential Revision: https://reviews.llvm.org/D67030
The file was modifiedclang/include/clang/Basic/SourceManager.h
The file was modifiedclang/lib/Basic/SourceManager.cpp
The file was modifiedclang/lib/Frontend/CompilerInstance.cpp
Commit 29d838310d4521fbd98741267b0a28cee5b56a36 by psteinfeld
[flang] Fix call to CHECK() on erroneous subroutine declaration

When processing declarations in resolve-names.cpp, we were returning a
symbol that had SubprogramName details to PushSubprogramScope(), which
expects a symbol with Subprogram details.

I adjusted the code and added a test.

Differential Revision: https://reviews.llvm.org/D89829
The file was modifiedflang/test/Semantics/resolve19.f90
The file was modifiedflang/lib/Semantics/resolve-names.cpp
Commit c565f09f4b0d908f51aaf4a841285f39ef93bc8c by Jonas Devlieghere
[lldb] Don't strip LLDB.framework on install

The framework build will run dsymutil after LLDB.framework is installed.
The file was modifiedlldb/cmake/caches/Apple-lldb-macOS.cmake
Commit 4aa97e3dacf3bdf5636fbf89dd8c64f1e4648065 by Duncan P. N. Exon Smith
SourceManager: Simplify early returns in ContentCache::getBufferOrNone, NFC

As suggested in the review for https://reviews.llvm.org/D89430, simplify
the logic for marking the buffer as invalid in the early return paths.

Differential Revision: https://reviews.llvm.org/D89722
The file was modifiedclang/lib/Basic/SourceManager.cpp
Commit acb33cba6d455dd96d82e2b746b2237f283a6416 by Cyndy Ishida
[llvm] Fix ODRViolations for VersionTuple YAML specializations NFC

It appears for Swift there was confusing errors when trying to parse APINotes, when libAPINotes and libInterfaceStub are linked, they both export symbol
`__ZN4llvm4yaml7yamlizeINS_12VersionTupleEEENSt3__19enable_ifIXsr16has_ScalarTraitsIT_EE5valueEvE4typeERNS0_2IOERS5_bRNS0_12EmptyContextE`, and discovered
same symbol defined within llvm-ifs.

This consolidates the boilerplate into YAMLTraits and defers the specific validation in reading the whole input.
fixes: rdar://problem/70450563

Reviewed By: phosek, dblaikie

Differential Revision: https://reviews.llvm.org/D89764
The file was modifiedllvm/include/llvm/Support/YAMLTraits.h
The file was modifiedllvm/unittests/InterfaceStub/ELFYAMLTest.cpp
The file was modifiedllvm/lib/InterfaceStub/TBEHandler.cpp
The file was modifiedllvm/lib/Support/YAMLTraits.cpp
The file was modifiedllvm/tools/llvm-ifs/llvm-ifs.cpp
Commit 829b9f6606af03e24d7715712e05c941d1c661ce by i
[test] Fix -fbasic-block-sections= test on Windows after D89500
The file was modifiedclang/test/CodeGen/basic-block-sections.c
Commit cbe0ee1a94d11bd32019920c8f55ebd58054542a by tu.da.wei
This is a test commit
The file was modifiedREADME.md
Commit aa07357c798c5f4d23e84b287a7ebcb1356f0af6 by tu.da.wei
Revert "This is a test commit"

This reverts commit cbe0ee1a94d11bd32019920c8f55ebd58054542a.
The file was modifiedREADME.md
Commit 2213a354b9d1cc0153ac20367e5397c66e660c32 by i
[Polly] Delete unused lambda capture after 7175cffb2133048018df74c1b49d1d4962ea18f2
The file was modifiedpolly/lib/Transform/ForwardOpTree.cpp