[all-commits] [llvm/llvm-project] 46a049: [DSE] Add stat for remaining stores after DSE.

Florian Hahn via All-commits all-commits at lists.llvm.org
Sat Apr 25 08:14:20 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 46a04940e85cea7d37d7791d18f940a49868c955
      https://github.com/llvm/llvm-project/commit/46a04940e85cea7d37d7791d18f940a49868c955
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2020-04-25 (Sat, 25 Apr 2020)

  Changed paths:
    M llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
    A llvm/test/Transforms/DeadStoreElimination/MSSA/stats.ll

  Log Message:
  -----------
  [DSE] Add stat for remaining stores after DSE.

Using the existing NumFastStores statistic can be misleading when
comparing the impact of DSE patches.

For example, consider the case where a store gets removed from a
function before it is inlined into another function. A less
powerful DSE might only remove the store from functions it has
been inlined into, which will result in more stores being removed, but
no difference in the actual number of stores after DSE.

The new stat provides the absolute number of stores surviving after
DSE.

Reviewers: dmgreen, bryant, asbirlea, jfb

Reviewed By: asbirlea

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




More information about the All-commits mailing list