[all-commits] [llvm/llvm-project] df4756: [DSE, MemorySSA] Check for underlying objects first.

Florian Hahn via All-commits all-commits at lists.llvm.org
Mon Aug 17 10:52:54 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: df4756ec6cd6e501bba645245c168f606adb4e67
      https://github.com/llvm/llvm-project/commit/df4756ec6cd6e501bba645245c168f606adb4e67
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2020-08-17 (Mon, 17 Aug 2020)

  Changed paths:
    M llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

  Log Message:
  -----------
  [DSE,MemorySSA] Check for underlying objects first.

isWriteAtEndOfFunction needs to check all memory uses of Def, which is
much more expensive than getting the underlying objects in practice.
Switch the call order, as recommended by the TODO, which was added as
per an earlier review.

This shaves off a bit of compile-time.




More information about the All-commits mailing list