Closed Quuxplusone closed 2 years ago
Bugzilla Link | PR52567 |
Status | RESOLVED FIXED |
Importance | P enhancement |
Reported by | Chris Burel (chrisburel@gmail.com) |
Reported on | 2021-11-19 16:39:06 -0800 |
Last modified on | 2021-11-19 22:26:58 -0800 |
Version | trunk |
Hardware | PC Linux |
CC | craig.topper@gmail.com, llvm-bugs@lists.llvm.org, llvm-dev@redking.me.uk, pengfei.wang@intel.com, spatel+llvm@rotateright.com |
Fixed by commit(s) | rGa4373f6753fa9aa89d39fbd4ec9e273f76459a58 |
Attachments | |
Blocks | |
Blocked by | |
See also |
I agree there is something funny going on here. I'll take a look.
Does this look better
rcpps .LCPI0_0(%rip), %xmm0
movaps .LCPI0_1(%rip), %xmm1 # xmm1 = [5.0E-1,2.5E-1,0.0E+0,0.0E+0]
subps %xmm0, %xmm1
andps .LCPI0_2(%rip), %xmm1
movaps .LCPI0_3(%rip), %xmm0 # xmm0 = [1.00000005E-3,1.00000005E-3,1.00000005E-3,1.00000005E-3]
cmpltps %xmm1, %xmm0
movmskps %xmm0, %ecx
xorl %eax, %eax
testb $3, %cl
sete %al
retq
I disabled this transform for this case in a4373f6753fa9aa89d39fbd4ec9e273f76459a58