[all-commits] [llvm/llvm-project] 3c12a5: [DSE] Add additional tests with free, regenerate c...

Florian Hahn via All-commits all-commits at lists.llvm.org
Sat Oct 31 13:11:09 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 3c12a5bdf14a43b5911eefe74b90447fe0d70f3a
      https://github.com/llvm/llvm-project/commit/3c12a5bdf14a43b5911eefe74b90447fe0d70f3a
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2020-10-31 (Sat, 31 Oct 2020)

  Changed paths:
    M llvm/test/Transforms/DeadStoreElimination/MSSA/free.ll

  Log Message:
  -----------
  [DSE] Add additional tests with free, regenerate check lines.

The new test cases are inspired by PR48036.


  Commit: aab71d44431eabfb65fbbc8a2930cb6d34b7e2be
      https://github.com/llvm/llvm-project/commit/aab71d44431eabfb65fbbc8a2930cb6d34b7e2be
  Author: Florian Hahn <flo at fhahn.com>
  Date:   2020-10-31 (Sat, 31 Oct 2020)

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

  Log Message:
  -----------
  [DSE] Use same logic as legacy impl to check if free kills a location.

This patch updates DSE + MemorySSA to use the same check as the legacy
implementation to determine if a location is killed by a free call.

This changes the existing behavior so that a free does not kill
locations before the start of the freed pointer.

This should fix PR48036.


Compare: https://github.com/llvm/llvm-project/compare/09662eeb46f7...aab71d44431e


More information about the All-commits mailing list