[all-commits] [llvm/llvm-project] e7afbe: [MemorySSA] Clear VisitedBlocks per query

whitneywhtsang via All-commits all-commits at lists.llvm.org
Fri Feb 18 12:36:36 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e7afbea8ca4e1c8239614aa37c2bd975172ddfa6
      https://github.com/llvm/llvm-project/commit/e7afbea8ca4e1c8239614aa37c2bd975172ddfa6
  Author: Whitney Tsang <whitneyt at ca.ibm.com>
  Date:   2022-02-18 (Fri, 18 Feb 2022)

  Changed paths:
    M llvm/lib/Analysis/MemorySSAUpdater.cpp
    M llvm/unittests/Analysis/MemorySSATest.cpp

  Log Message:
  -----------
  [MemorySSA] Clear VisitedBlocks per query

The problem can be shown from the newly added test case.
There are two invocations to MemorySSAUpdater::moveToPlace, and the
internal data structure VisitedBlocks is changed in the first
invocation, and reused in the second invocation. In between the two
invocations, there is a change to the CFG, and MemorySSAUpdater is
notified about the change.

Reviewed By: asbirlea

Differential Revision: https://reviews.llvm.org/D119898




More information about the All-commits mailing list