[PATCH] D127964: [DCE] Eliminate no-op atan and atan2 calls

[Mohammed Nurul Hoque via Phabricator via llvm-commits]

mohammed-nurulhoque added inline comments.


================
Comment at: llvm/test/Transforms/EarlyCSE/atan.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -early-cse -S < %s | FileCheck %s
+
----------------
spatel wrote:
> Unless there's a reason/difference, I'd expect tests like this to be in the folder tests/Transforms/InstSimplify/ConstProp with similar tests for mathlib calls and use "RUN: opt -passes=instsimplify..."
instsimplify does the "replacing the use of the call return value with a constant" part, but does not do the "eliminating the now redundant call" part. This patch adds the elimination not the replacement with constant.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127964/new/

https://reviews.llvm.org/D127964