tree 7993c7e49bca8d261ad2fff8b83eea9f6f8c0656
parent b7126bab3a192d7f7f86f2ab6975fc911f85858b
author Nicolas Capens <capn@google.com> 1652246416 -0400
committer Nicolas Capens <nicolascapens@google.com> 1652313709 +0000

Revert rcpss/rsqrtss instrumentation to default propagation

_mm_round_ss() takes two vector arguments, while _mm_rcp_ss() takes only
one, despite both underlying instructions taking one source operand.
LLVM uses the C++ intrinsics behavior, so x86_sse_rcp_ss and
x86_sse_rsqrt_ss don't support the shuffle operation between the first
and second argument that is performed by handleUnarySdSsIntrinsic().

Instead the behavior of the fallback path, which calls
maybeHandleSimpleNomemIntrinsic() and propagates the shadow, was fine.

Bug: b/172238865
Change-Id: I1bfbcaf518dd90f5a113cdef2b75d1a602e43d90
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/65808
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Sean Risser <srisser@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>