FailedChanges

Summary

  1. Determine endianness at a time when it doesn't inadvertantly clear gdb's wrap_buffer via gdb.execute. Summary: I haven't managed a small reproduction for this bug, it involves complicated and deeply nested data structures with a wide variety of pretty printers. But in general, we shouldn't be combining gdb's command line interface (via gdb.execute) with pretty-printers. Subscribers: christof, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D68306
  2. Revert r373172 "[X86] Add custom isel logic to match VPTERNLOG from 2 logic ops." This seems to be causing some performance regresions that I'm trying to investigate. One thing that stands out is that this transform can increase the live range of the operands of the earlier logic op. This can be bad for register allocation. If there are two logic op inputs we should really combine the one that is closest, but SelectionDAG doesn't have a good way to do that. Maybe we need to do this as a basic block transform in Machine IR.
  3. [clang][OpenMP][NFC] #include GlobalDecl.h to avoid incomplete class type
  4. CGVTables - silence static analyzer getAs<FunctionProtoType> null dereference warnings. NFCI. The static analyzer is warning about potential null dereferences, but we should be able to use castAs<FunctionProtoType> directly and if not assert will fire for us.
  5. [X86] convertToThreeAddress, make sure second operand of SUB32ri is really an immediate before calling getImm(). It might be a symbol instead. We can't fold those since we can't negate them. Similar for other SUB with immediates. Fixes PR43529.
  6. CGExprAgg - remove duplicate code. NFCI. Remove duplicate getAs<> call, avoiding a clang static analyzer null dereference warning.
  7. [FileCheck] Move private interface to its own header Summary: Most of the class definition in llvm/include/llvm/Support/FileCheck.h are actually implementation details that should not be relied upon. This commit moves all of it in a new header file under llvm/lib/Support/FileCheck. It also takes advantage of the code movement to put the code into a new llvm::filecheck namespace. Reviewers: jhenderson, chandlerc, jdenny, probinson, grimar, arichardson, rnk Subscribers: hiraditya, llvm-commits, probinson, dblaikie, grimar, arichardson, tra, rnk, kristina, hfinkel, rogfer01, JonChesterfield Tags: #llvm Differential Revision: https://reviews.llvm.org/D67649
  8. [BypassSlowDivision][CodeGenPrepare] avoid crashing on unused code (PR43514) https://bugs.llvm.org/show_bug.cgi?id=43514
  9. gn build: Merge r373392
  10. [clang-tidy] Rename objc-avoid-spinlock check to darwin-avoid-spinlock Summary: OSSpinLock* are Apple/Darwin functions, but were previously located with ObjC checks as those were most closely tied to Apple platforms before. Now that there's a specific Darwin module, relocating the check there. This change was prepared by running rename_check.py. Contributed By: mwyman Reviewers: stephanemoore, dmaclach Reviewed By: stephanemoore Subscribers: Eugene.Zelenko, mgorny, xazax.hun, cfe-commits Tags: #clang-tools-extra, #clang, #llvm Differential Revision: https://reviews.llvm.org/D68148
  11. [ASan][NFC] Address remaining comments for https://reviews.llvm.org/D68287 I submitted that patch after I got the LGTM, but the comments didn't appear until after I submitted the change. This adds `const` to the constructor argument and makes it a pointer.
  12. [NFC] Fix typos in libc++ documentation
  13. [ASan] Make GlobalsMD member a const reference. PR42924 points out that copying the GlobalsMetadata type during construction of AddressSanitizer can result in exteremely lengthened build times for translation units that have many globals. This can be addressed by just making the GlobalsMD member in AddressSanitizer a reference to avoid the copy. The GlobalsMetadata type is already passed to the constructor as a reference anyway. Differential Revision: https://reviews.llvm.org/D68287
  14. [clang-format] [PR43372] - clang-format shows replacements in DOS files when no replacement is needed Summary: This is a patch to fix PR43372 (https://bugs.llvm.org/show_bug.cgi?id=43372) - clang-format can't format file with includes, ( which really keep providing replacements for already sorted headers.) A similar issue was addressed by @krasimir in {D60199}, however, this seemingly only prevented the issue when the files being formatted did not contain windows line endings (\r\n) It's possible this is related to https://twitter.com/StephanTLavavej/status/1176722938243895296 given who @STL_MSFT works for! As people often used the existence of replacements to determine if a file needs clang-formatting, this is probably pretty important for windows users There may be a better way of comparing 2 strings and ignoring \r (which appear in both Results and Code), I couldn't choose between this idiom or the copy_if approach, but I'm happy to change it to whatever people consider more performant. Reviewers: krasimir, klimek, owenpan, ioeric Reviewed By: krasimir Subscribers: cfe-commits, STL_MSFT, krasimir Tags: #clang-format, #clang, #clang-tools-extra Differential Revision: https://reviews.llvm.org/D68227
  15. [OPENMP50]Initial codegen for declare variant implementation vendor. Initial implementation of global aliases emission for the declare variant pragma with implementation vendor context selector set.
  16. [DDG] Data Dependence Graph - Root Node Summary: This patch adds Root Node to the DDG. The purpose of the root node is to create a single entry node that allows graph walk iterators to iterate through all nodes of the graph, making sure that no node is left unvisited during a graph walk (eg. SCC or DFS). Once the DDG is fully constructed it will have exactly one root node. Every node in the graph is reachable from the root. The algorithm for connecting the root node is based on depth-first-search that keeps track of visited nodes to try to avoid creating unnecessary edges. Authored By: bmahjour Reviewer: Meinersbur, fhahn, myhsu, xtian, dmgreen, kbarton, jdoerfert Reviewed By: Meinersbur Subscribers: ychen, arphaman, simoll, a.elovikov, mgorny, hiraditya, jfb, wuzish, llvm-commits, jsji, Whitney, etiotto, ppc-slack Tag: #llvm Differential Revision: https://reviews.llvm.org/D67970
  17. [libc++] Re-apply workaround for D63883 In reality, this workaround is for the fact that LIBCXX_CXX_ABI=libcxxabi can't be specified on Linux, since libc++abi isn't shipped with the system. Since the build bots explicitly specify LIBCXX_CXX_ABI=libcxxabi, they fail unless we apply the workaround.
  18. [libc++] Remove workaround for D63883 I tried applying D63883 three times and could never get around to making it work. I'm giving up on that for now, but soon this should be irrelevant anyway since all builds will move to the monorepo (where we're always using the in-tree libc++abi unless explicitly specified otherwise).
  19. [MemorySSA] Check for unreachable blocks when getting last definition. If a single predecessor is found, still check if the block is unreachable. The test that found this had a self loop unreachable block. Resolves PR43493.
  20. Add a missing pass in ARM O3 pipeline
  21. [MemorySSA] Update last_access_in_block check. The check for "was there an access in this block" should be: is the last access in this block and is it not a newly inserted phi. Resolves new test in PR43438. Also fix a typo when simplifying trivial Phis to match the comment.
Revision 373402 by saugustine:
Determine endianness at a time when it doesn't inadvertantly clear gdb's wrap_buffer via gdb.execute.

Summary:
I haven't managed a small reproduction for this bug, it involves
complicated and deeply nested data structures with a wide variety
of pretty printers. But in general, we shouldn't be combining
gdb's command line interface (via gdb.execute) with pretty-printers.

Subscribers: christof, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D68306
Change TypePath in RepositoryPath in Workspace
The file was modified/libcxx/trunk/utils/gdb/libcxx/printers.pylibcxx.src/utils/gdb/libcxx/printers.py
Revision 373401 by ctopper:
Revert r373172 "[X86] Add custom isel logic to match VPTERNLOG from 2 logic ops."

This seems to be causing some performance regresions that I'm
trying to investigate.

One thing that stands out is that this transform can increase
the live range of the operands of the earlier logic op. This
can be bad for register allocation. If there are two logic
op inputs we should really combine the one that is closest, but
SelectionDAG doesn't have a good way to do that. Maybe we need
to do this as a basic block transform in Machine IR.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86ISelDAGToDAG.cppllvm.src/lib/Target/X86/X86ISelDAGToDAG.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-cvt.llllvm.src/test/CodeGen/X86/avx512-cvt.ll
The file was modified/llvm/trunk/test/CodeGen/X86/avx512-gfni-intrinsics.llllvm.src/test/CodeGen/X86/avx512-gfni-intrinsics.ll
The file was modified/llvm/trunk/test/CodeGen/X86/machine-combiner-int-vec.llllvm.src/test/CodeGen/X86/machine-combiner-int-vec.ll
The file was modified/llvm/trunk/test/CodeGen/X86/midpoint-int-vec-128.llllvm.src/test/CodeGen/X86/midpoint-int-vec-128.ll
The file was modified/llvm/trunk/test/CodeGen/X86/midpoint-int-vec-256.llllvm.src/test/CodeGen/X86/midpoint-int-vec-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/midpoint-int-vec-512.llllvm.src/test/CodeGen/X86/midpoint-int-vec-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/sadd_sat_vec.llllvm.src/test/CodeGen/X86/sadd_sat_vec.ll
The file was modified/llvm/trunk/test/CodeGen/X86/ssub_sat_vec.llllvm.src/test/CodeGen/X86/ssub_sat_vec.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vec_int_to_fp.llllvm.src/test/CodeGen/X86/vec_int_to_fp.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshl-256.llllvm.src/test/CodeGen/X86/vector-fshl-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshl-512.llllvm.src/test/CodeGen/X86/vector-fshl-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshl-rot-256.llllvm.src/test/CodeGen/X86/vector-fshl-rot-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshl-rot-512.llllvm.src/test/CodeGen/X86/vector-fshl-rot-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshr-256.llllvm.src/test/CodeGen/X86/vector-fshr-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshr-512.llllvm.src/test/CodeGen/X86/vector-fshr-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshr-rot-256.llllvm.src/test/CodeGen/X86/vector-fshr-rot-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-fshr-rot-512.llllvm.src/test/CodeGen/X86/vector-fshr-rot-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-idiv-sdiv-512.llllvm.src/test/CodeGen/X86/vector-idiv-sdiv-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-rotate-128.llllvm.src/test/CodeGen/X86/vector-rotate-128.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-rotate-256.llllvm.src/test/CodeGen/X86/vector-rotate-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-rotate-512.llllvm.src/test/CodeGen/X86/vector-rotate-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shift-ashr-128.llllvm.src/test/CodeGen/X86/vector-shift-ashr-128.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shift-ashr-256.llllvm.src/test/CodeGen/X86/vector-shift-ashr-256.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shift-ashr-512.llllvm.src/test/CodeGen/X86/vector-shift-ashr-512.ll
The file was modified/llvm/trunk/test/CodeGen/X86/vector-shift-ashr-sub128.llllvm.src/test/CodeGen/X86/vector-shift-ashr-sub128.ll
Revision 373400 by rupprecht:
[clang][OpenMP][NFC] #include GlobalDecl.h to avoid incomplete class type
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/CodeGen/CGOpenMPRuntime.hclang.src/lib/CodeGen/CGOpenMPRuntime.h
Revision 373398 by rksimon:
CGVTables - silence static analyzer getAs<FunctionProtoType> null dereference warnings. NFCI.

The static analyzer is warning about potential null dereferences, but we should be able to use castAs<FunctionProtoType> directly and if not assert will fire for us.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/CodeGen/CGVTables.cppclang.src/lib/CodeGen/CGVTables.cpp
Revision 373397 by ctopper:
[X86] convertToThreeAddress, make sure second operand of SUB32ri is really an immediate before calling getImm().

It might be a symbol instead. We can't fold those since we can't
negate them.

Similar for other SUB with immediates.

Fixes PR43529.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/X86/X86InstrInfo.cppllvm.src/lib/Target/X86/X86InstrInfo.cpp
The file was added/llvm/trunk/test/CodeGen/X86/pr43529.llllvm.src/test/CodeGen/X86/pr43529.ll
Revision 373396 by rksimon:
CGExprAgg - remove duplicate code. NFCI.

Remove duplicate getAs<> call, avoiding a clang static analyzer null dereference warning.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/CodeGen/CGExprAgg.cppclang.src/lib/CodeGen/CGExprAgg.cpp
Revision 373395 by thopre:
[FileCheck] Move private interface to its own header

Summary:
Most of the class definition in llvm/include/llvm/Support/FileCheck.h
are actually implementation details that should not be relied upon. This
commit moves all of it in a new header file under
llvm/lib/Support/FileCheck. It also takes advantage of the code movement
to put the code into a new llvm::filecheck namespace.

Reviewers: jhenderson, chandlerc, jdenny, probinson, grimar, arichardson, rnk

Subscribers: hiraditya, llvm-commits, probinson, dblaikie, grimar, arichardson, tra, rnk, kristina, hfinkel, rogfer01, JonChesterfield

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D67649
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Support/FileCheck.hllvm.src/include/llvm/Support/FileCheck.h
The file was modified/llvm/trunk/lib/Support/FileCheck.cppllvm.src/lib/Support/FileCheck.cpp
The file was added/llvm/trunk/lib/Support/FileCheckImpl.hllvm.src/lib/Support/FileCheckImpl.h
The file was modified/llvm/trunk/unittests/Support/FileCheckTest.cppllvm.src/unittests/Support/FileCheckTest.cpp
Revision 373394 by spatel:
[BypassSlowDivision][CodeGenPrepare] avoid crashing on unused code (PR43514)

https://bugs.llvm.org/show_bug.cgi?id=43514
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Utils/BypassSlowDivision.cppllvm.src/lib/Transforms/Utils/BypassSlowDivision.cpp
The file was modified/llvm/trunk/test/CodeGen/X86/bypass-slow-division-64.llllvm.src/test/CodeGen/X86/bypass-slow-division-64.ll
Revision 373393 by gnsyncbot:
gn build: Merge r373392
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/utils/gn/secondary/clang-tools-extra/clang-tidy/darwin/BUILD.gnllvm.src/utils/gn/secondary/clang-tools-extra/clang-tidy/darwin/BUILD.gn
The file was modified/llvm/trunk/utils/gn/secondary/clang-tools-extra/clang-tidy/objc/BUILD.gnllvm.src/utils/gn/secondary/clang-tools-extra/clang-tidy/objc/BUILD.gn
Revision 373392 by stephanemoore:
[clang-tidy] Rename objc-avoid-spinlock check to darwin-avoid-spinlock

Summary:
OSSpinLock* are Apple/Darwin functions, but were previously located with ObjC checks as those were most closely tied to Apple platforms before.

Now that there's a specific Darwin module, relocating the check there.

This change was prepared by running rename_check.py.

Contributed By: mwyman

Reviewers: stephanemoore, dmaclach

Reviewed By: stephanemoore

Subscribers: Eugene.Zelenko, mgorny, xazax.hun, cfe-commits

Tags: #clang-tools-extra, #clang, #llvm

Differential Revision: https://reviews.llvm.org/D68148
Change TypePath in RepositoryPath in Workspace
The file was added/clang-tools-extra/trunk/clang-tidy/darwin/AvoidSpinlockCheck.cppclang-tools-extra.src/clang-tidy/darwin/AvoidSpinlockCheck.cpp
The file was added/clang-tools-extra/trunk/clang-tidy/darwin/AvoidSpinlockCheck.hclang-tools-extra.src/clang-tidy/darwin/AvoidSpinlockCheck.h
The file was modified/clang-tools-extra/trunk/clang-tidy/darwin/CMakeLists.txtclang-tools-extra.src/clang-tidy/darwin/CMakeLists.txt
The file was modified/clang-tools-extra/trunk/clang-tidy/darwin/DarwinTidyModule.cppclang-tools-extra.src/clang-tidy/darwin/DarwinTidyModule.cpp
The file was removed/clang-tools-extra/trunk/clang-tidy/objc/AvoidSpinlockCheck.cppclang-tools-extra.src/clang-tidy/objc/AvoidSpinlockCheck.cpp
The file was removed/clang-tools-extra/trunk/clang-tidy/objc/AvoidSpinlockCheck.hclang-tools-extra.src/clang-tidy/objc/AvoidSpinlockCheck.h
The file was modified/clang-tools-extra/trunk/clang-tidy/objc/CMakeLists.txtclang-tools-extra.src/clang-tidy/objc/CMakeLists.txt
The file was modified/clang-tools-extra/trunk/clang-tidy/objc/ObjCTidyModule.cppclang-tools-extra.src/clang-tidy/objc/ObjCTidyModule.cpp
The file was modified/clang-tools-extra/trunk/docs/ReleaseNotes.rstclang-tools-extra.src/docs/ReleaseNotes.rst
The file was added/clang-tools-extra/trunk/docs/clang-tidy/checks/darwin-avoid-spinlock.rstclang-tools-extra.src/docs/clang-tidy/checks/darwin-avoid-spinlock.rst
The file was modified/clang-tools-extra/trunk/docs/clang-tidy/checks/list.rstclang-tools-extra.src/docs/clang-tidy/checks/list.rst
The file was removed/clang-tools-extra/trunk/docs/clang-tidy/checks/objc-avoid-spinlock.rstclang-tools-extra.src/docs/clang-tidy/checks/objc-avoid-spinlock.rst
The file was added/clang-tools-extra/trunk/test/clang-tidy/darwin-avoid-spinlock.mclang-tools-extra.src/test/clang-tidy/darwin-avoid-spinlock.m
The file was removed/clang-tools-extra/trunk/test/clang-tidy/objc-avoid-spinlock.mclang-tools-extra.src/test/clang-tidy/objc-avoid-spinlock.m
Revision 373391 by leonardchan:
[ASan][NFC] Address remaining comments for https://reviews.llvm.org/D68287

I submitted that patch after I got the LGTM, but the comments didn't
appear until after I submitted the change. This adds `const` to the
constructor argument and makes it a pointer.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cppllvm.src/lib/Transforms/Instrumentation/AddressSanitizer.cpp
Revision 373390 by Louis Dionne:
[NFC] Fix typos in libc++ documentation
Change TypePath in RepositoryPath in Workspace
The file was modified/libcxx/trunk/docs/BuildingLibcxx.rstlibcxx.src/docs/BuildingLibcxx.rst
Revision 373389 by leonardchan:
[ASan] Make GlobalsMD member a const reference.

PR42924 points out that copying the GlobalsMetadata type during
construction of AddressSanitizer can result in exteremely lengthened
build times for translation units that have many globals. This can be addressed
by just making the GlobalsMD member in AddressSanitizer a reference to
avoid the copy. The GlobalsMetadata type is already passed to the
constructor as a reference anyway.

Differential Revision: https://reviews.llvm.org/D68287
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/Instrumentation/AddressSanitizer.cppllvm.src/lib/Transforms/Instrumentation/AddressSanitizer.cpp
Revision 373388 by paulhoad:
[clang-format] [PR43372] - clang-format shows replacements in DOS files when no replacement is needed

Summary:
This is a patch to fix PR43372 (https://bugs.llvm.org/show_bug.cgi?id=43372) - clang-format can't format file with includes, ( which really keep providing replacements for already sorted headers.)

A similar issue was addressed by @krasimir in {D60199}, however, this seemingly only prevented the issue when the files being formatted did not contain windows line endings (\r\n)

It's possible this is related to https://twitter.com/StephanTLavavej/status/1176722938243895296 given who @STL_MSFT  works for!

As people often used the existence of replacements to determine if a file needs clang-formatting, this is probably pretty important for windows users

There may be a better way of comparing 2 strings and ignoring \r (which appear in both Results and Code), I couldn't choose between this idiom or the copy_if approach, but I'm happy to change it to whatever people consider more performant.

Reviewers: krasimir, klimek, owenpan, ioeric

Reviewed By: krasimir

Subscribers: cfe-commits, STL_MSFT, krasimir

Tags: #clang-format, #clang, #clang-tools-extra

Differential Revision: https://reviews.llvm.org/D68227
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Format/Format.cppclang.src/lib/Format/Format.cpp
The file was modified/cfe/trunk/unittests/Format/SortImportsTestJava.cppclang.src/unittests/Format/SortImportsTestJava.cpp
The file was modified/cfe/trunk/unittests/Format/SortIncludesTest.cppclang.src/unittests/Format/SortIncludesTest.cpp
Revision 373387 by abataev:
[OPENMP50]Initial codegen for declare variant implementation vendor.

Initial implementation of global aliases emission for the declare
variant pragma with implementation vendor context selector set.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/CodeGen/CGOpenMPRuntime.cppclang.src/lib/CodeGen/CGOpenMPRuntime.cpp
The file was modified/cfe/trunk/lib/CodeGen/CGOpenMPRuntime.hclang.src/lib/CodeGen/CGOpenMPRuntime.h
The file was modified/cfe/trunk/lib/CodeGen/CodeGenModule.cppclang.src/lib/CodeGen/CodeGenModule.cpp
The file was added/cfe/trunk/test/OpenMP/declare_variant_implementation_vendor_codegen.cppclang.src/test/OpenMP/declare_variant_implementation_vendor_codegen.cpp
Revision 373386 by bmahjour:
[DDG] Data Dependence Graph - Root Node

Summary:
This patch adds Root Node to the DDG. The purpose of the root node is to create a single entry node that allows graph walk iterators to iterate through all nodes of the graph, making sure that no node is left unvisited during a graph walk (eg. SCC or DFS). Once the DDG is fully constructed it will have exactly one root node. Every node in the graph is reachable from the root. The algorithm for connecting the root node is based on depth-first-search that keeps track of visited nodes to try to avoid creating unnecessary edges.

Authored By: bmahjour

Reviewer: Meinersbur, fhahn, myhsu, xtian, dmgreen, kbarton, jdoerfert

Reviewed By: Meinersbur

Subscribers: ychen, arphaman, simoll, a.elovikov, mgorny, hiraditya, jfb, wuzish, llvm-commits, jsji, Whitney, etiotto, ppc-slack

Tag: #llvm

Differential Revision: https://reviews.llvm.org/D67970
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Analysis/DDG.hllvm.src/include/llvm/Analysis/DDG.h
The file was modified/llvm/trunk/include/llvm/Analysis/DependenceGraphBuilder.hllvm.src/include/llvm/Analysis/DependenceGraphBuilder.h
The file was modified/llvm/trunk/lib/Analysis/DDG.cppllvm.src/lib/Analysis/DDG.cpp
The file was modified/llvm/trunk/lib/Analysis/DependenceGraphBuilder.cppllvm.src/lib/Analysis/DependenceGraphBuilder.cpp
The file was added/llvm/trunk/test/Analysis/DDG/root-node.llllvm.src/test/Analysis/DDG/root-node.ll
Revision 373385 by Louis Dionne:
[libc++] Re-apply workaround for D63883

In reality, this workaround is for the fact that LIBCXX_CXX_ABI=libcxxabi
can't be specified on Linux, since libc++abi isn't shipped with the system.
Since the build bots explicitly specify LIBCXX_CXX_ABI=libcxxabi, they fail
unless we apply the workaround.
Change TypePath in RepositoryPath in Workspace
The file was modified/libcxx/trunk/CMakeLists.txtlibcxx.src/CMakeLists.txt
Revision 373384 by Louis Dionne:
[libc++] Remove workaround for D63883

I tried applying D63883 three times and could never get around to
making it work. I'm giving up on that for now, but soon this should
be irrelevant anyway since all builds will move to the monorepo
(where we're always using the in-tree libc++abi unless explicitly
specified otherwise).
Change TypePath in RepositoryPath in Workspace
The file was modified/libcxx/trunk/CMakeLists.txtlibcxx.src/CMakeLists.txt
Revision 373383 by asbirlea:
[MemorySSA] Check for unreachable blocks when getting last definition.

If a single predecessor is found, still check if the block is
unreachable. The test that found this had a self loop unreachable block.
Resolves PR43493.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Analysis/MemorySSAUpdater.cppllvm.src/lib/Analysis/MemorySSAUpdater.cpp
The file was added/llvm/trunk/test/Analysis/MemorySSA/pr43493.llllvm.src/test/Analysis/MemorySSA/pr43493.ll
Revision 373382 by kuhar:
Add a missing pass in ARM O3 pipeline
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/CodeGen/ARM/O3-pipeline.llllvm.src/test/CodeGen/ARM/O3-pipeline.ll
Revision 373380 by asbirlea:
[MemorySSA] Update last_access_in_block check.

The check for "was there an access in this block" should be: is the last
access in this block and is it not a newly inserted phi.
Resolves new test in PR43438.

Also fix a typo when simplifying trivial Phis to match the comment.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Analysis/MemorySSAUpdater.cppllvm.src/lib/Analysis/MemorySSAUpdater.cpp
The file was modified/llvm/trunk/test/Analysis/MemorySSA/pr43438.llllvm.src/test/Analysis/MemorySSA/pr43438.ll