diff options
| author | Bob Wilson <bob.wilson@apple.com> | 2012-08-03 23:29:17 +0000 |
|---|---|---|
| committer | Bob Wilson <bob.wilson@apple.com> | 2012-08-03 23:29:17 +0000 |
| commit | 53624a2df557b4a24f2ee98cfce1a69bf83243af (patch) | |
| tree | ce940e202cc56262d4a9c96df2ac551def98d380 /test/CodeGen/X86/fabs.ll | |
| parent | 6ac8066ae49a5e9910f24e08af0b168210270946 (diff) | |
| download | external_llvm-53624a2df557b4a24f2ee98cfce1a69bf83243af.zip external_llvm-53624a2df557b4a24f2ee98cfce1a69bf83243af.tar.gz external_llvm-53624a2df557b4a24f2ee98cfce1a69bf83243af.tar.bz2 | |
Refactor and check "onlyReadsMemory" before optimizing builtins.
This patch is mostly just refactoring a bunch of copy-and-pasted code, but
it also adds a check that the call instructions are readnone or readonly.
That check was already present for sin, cos, sqrt, log2, and exp2 calls, but
it was missing for the rest of the builtins being handled in this code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@161282 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CodeGen/X86/fabs.ll')
| -rw-r--r-- | test/CodeGen/X86/fabs.ll | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/CodeGen/X86/fabs.ll b/test/CodeGen/X86/fabs.ll index 07a8d91..2074f04 100644 --- a/test/CodeGen/X86/fabs.ll +++ b/test/CodeGen/X86/fabs.ll @@ -11,7 +11,7 @@ declare x86_fp80 @fabsl(x86_fp80) ; UNSAFE: test1: ; NOOPT: test1: define float @test1(float %X) { - %Y = call float @fabsf(float %X) + %Y = call float @fabsf(float %X) readnone ret float %Y } ; CHECK: {{^[ \t]+fabs$}} @@ -42,7 +42,7 @@ define double @test2(double %X) { ; UNSAFE: test3: ; NOOPT: test3: define x86_fp80 @test3(x86_fp80 %X) { - %Y = call x86_fp80 @fabsl(x86_fp80 %X) + %Y = call x86_fp80 @fabsl(x86_fp80 %X) readnone ret x86_fp80 %Y } ; CHECK: {{^[ \t]+fabs$}} |