[all-commits] [llvm/llvm-project] 81d69e: [DSE] Call isRemovable() after getLocForWriteEx() ...

Nikita Popov via All-commits all-commits at lists.llvm.org
Fri Dec 24 01:01:40 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 81d69e1bda9e4b6a83f29ba1f614e43ab4700972
      https://github.com/llvm/llvm-project/commit/81d69e1bda9e4b6a83f29ba1f614e43ab4700972
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2021-12-24 (Fri, 24 Dec 2021)

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

  Log Message:
  -----------
  [DSE] Call isRemovable() after getLocForWriteEx() (NFCI)

The only non-trivial change here is that the isReadClobber()
check for redundant stores is now on the DefLoc, not the
UpperLoc. This is semantically the right location to use, though
in practice it makes no difference (the locations are either the
same, or the def inst does not read).




More information about the All-commits mailing list