[PATCH] D47993: [x86] fix mappings of cvttp2si x86 intrinsics to x86-specific nodes and isel patterns (PR37551)

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In https://reviews.llvm.org/D47993#1129179, @craig.topper wrote:

> I think I've fixed the missing load folding in r334460.
>
> For avx512 I think we should also update those table entries to use X86ISD::CVTTP2SI


Is it correct that those updates would be independent of the current patch? If so, I'd like to move ahead with this part as-is.

I don't know anything about AVX512, so I don't want to jeopardize these fixes (assuming they're good) with more diffs.

Alternatively, if someone wants to commandeer this and add the AVX512 part, I'll gladly go hack on other stuff. :)


https://reviews.llvm.org/D47993