diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2019-11-26 17:35:10 -0500 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2019-11-26 17:35:10 -0500 |
| commit | e177c5a00da34ba61b762e2b32bd96e33b0c10b4 (patch) | |
| tree | 9a5c874676bb8f9f6947977383f494a9f400a9bd | |
| parent | 48a3a1e090611d1a71cb3c027e9316d048a67324 (diff) | |
| download | bcm5719-llvm-e177c5a00da34ba61b762e2b32bd96e33b0c10b4.tar.gz bcm5719-llvm-e177c5a00da34ba61b762e2b32bd96e33b0c10b4.zip | |
[InstSimplify] fold copysign with same args to the arg
This is correct for any value including NaN/inf.
We don't have this fold directly in the backend either,
but x86 manages to get it after converting things to bitops.
| -rw-r--r-- | llvm/lib/Analysis/InstructionSimplify.cpp | 5 | ||||
| -rw-r--r-- | llvm/test/Transforms/InstSimplify/call.ll | 6 |
2 files changed, 7 insertions, 4 deletions
diff --git a/llvm/lib/Analysis/InstructionSimplify.cpp b/llvm/lib/Analysis/InstructionSimplify.cpp index d997acb365c..7942cb09e84 100644 --- a/llvm/lib/Analysis/InstructionSimplify.cpp +++ b/llvm/lib/Analysis/InstructionSimplify.cpp @@ -5086,6 +5086,11 @@ static Value *simplifyBinaryIntrinsic(Function *F, Value *Op0, Value *Op1, return Op0; } break; + case Intrinsic::copysign: + // copysign X, X --> X + if (Op0 == Op1) + return Op0; + break; case Intrinsic::maxnum: case Intrinsic::minnum: case Intrinsic::maximum: diff --git a/llvm/test/Transforms/InstSimplify/call.ll b/llvm/test/Transforms/InstSimplify/call.ll index b0f35ab3715..4736adb972d 100644 --- a/llvm/test/Transforms/InstSimplify/call.ll +++ b/llvm/test/Transforms/InstSimplify/call.ll @@ -926,8 +926,7 @@ declare <2 x double> @llvm.copysign.v2f64(<2 x double>, <2 x double>) define float @copysign_same_operand(float %x) { ; CHECK-LABEL: @copysign_same_operand( -; CHECK-NEXT: [[R:%.*]] = call float @llvm.copysign.f32(float [[X:%.*]], float [[X]]) -; CHECK-NEXT: ret float [[R]] +; CHECK-NEXT: ret float [[X:%.*]] ; %r = call float @llvm.copysign.f32(float %x, float %x) ret float %r @@ -935,8 +934,7 @@ define float @copysign_same_operand(float %x) { define <2 x double> @copysign_same_operand_vec(<2 x double> %x) { ; CHECK-LABEL: @copysign_same_operand_vec( -; CHECK-NEXT: [[R:%.*]] = call <2 x double> @llvm.copysign.v2f64(<2 x double> [[X:%.*]], <2 x double> [[X]]) -; CHECK-NEXT: ret <2 x double> [[R]] +; CHECK-NEXT: ret <2 x double> [[X:%.*]] ; %r = call <2 x double> @llvm.copysign.v2f64(<2 x double> %x, <2 x double> %x) ret <2 x double> %r |
