[all-commits] [llvm/llvm-project] 8aa9ab: [ADCE] Only remove debug intrinsics if non debug i...

[mikaelholmen via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8aa9ab336889ae2eb8e4188036faeb151379ab7b
      https://github.com/llvm/llvm-project/commit/8aa9ab336889ae2eb8e4188036faeb151379ab7b
  Author: Mikael Holmen <mikael.holmen at ericsson.com>
  Date:   2023-03-03 (Fri, 03 Mar 2023)

  Changed paths:
    M llvm/lib/Transforms/Scalar/ADCE.cpp
    A llvm/test/Transforms/ADCE/debug-info-intrinsic-2.ll
    M llvm/test/Transforms/ADCE/debug-info-intrinsic.ll
    M llvm/test/Transforms/MergeFunc/mergefunc-preserve-debug-info.ll

  Log Message:
  -----------
  [ADCE] Only remove debug intrinsics if non debug instructions are removed

We now limit ADCE to only remove debug intrinsics if it does something else
that would invalidate cached analyses anyway.
As we've seen in
 https://github.com/llvm/llvm-project/issues/58285
throwing away cached analysis info when only debug instructions are removed
can lead to different code when debug info is present or not present.

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