[all-commits] [llvm/llvm-project] 14a2bb: [MemorySSA] Combine verifications.

Alina Sbirlea via All-commits all-commits at lists.llvm.org
Mon Nov 25 16:09:48 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 14a2bbb1ff9439b798f29e5ae63a6f9240a72983
      https://github.com/llvm/llvm-project/commit/14a2bbb1ff9439b798f29e5ae63a6f9240a72983
  Author: Alina Sbirlea <asbirlea at google.com>
  Date:   2019-11-25 (Mon, 25 Nov 2019)

  Changed paths:
    M llvm/include/llvm/Analysis/MemorySSA.h
    M llvm/lib/Analysis/MemorySSA.cpp

  Log Message:
  -----------
  [MemorySSA] Combine verifications.

Summary:
Combine three verification methods into one to improve compile time when asserts are enabled.
Motivated by PR44066.

Sample change of timings on testcase in PR44066 (release+asserts):
MSSA off or verification disabled: 1.13s.
MSSA on (ToT): 2.48s.
With patch: 2.03s.
With enabling DefUses after combining Domination+Ordering: 2.6s.
After also combining DefUses with Domination+Ordering: 2.06s (candidate to be taken out of EXPENSIVE_CHECKS).

Subscribers: Prazek, hiraditya, george.burgess.iv, sanjoy.google, llvm-commits

Tags: #llvm

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




More information about the All-commits mailing list