This is an archive of the discontinued LLVM Phabricator instance.

[X86][AVX512DQ] add scalar fpclass
ClosedPublic

Authored by AsafBadouh on Oct 15 2015, 4:53 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

AsafBadouh updated this revision to Diff 37472.Oct 15 2015, 4:53 AM
AsafBadouh retitled this revision from to [X86][AVX512DQ] add scalar fpclass.
AsafBadouh updated this object.
AsafBadouh updated this revision to Diff 37475.Oct 15 2015, 4:57 AM
AsafBadouh updated this object.
AsafBadouh added reviewers: delena, igorb.
AsafBadouh added a subscriber: llvm-commits.

some clean up

replace "ISD::SIGN_EXTEND_INREG" in "ISD::SIGN_EXTEND"

delena added inline comments.Oct 18 2015, 12:23 AM
../commit/llvm_clean/lib/Target/X86/X86ISelLowering.cpp
16301 ↗(On Diff #37696)

I did not mean that you put SIGN_EXTEND on ANY_EXTEND. It is just meaningless.
Is this possible:
return DAG.getNode(ISD::SIGN_EXTEND, dl, MVT::i8,FPclassMask ) ?

AsafBadouh updated this revision to Diff 37700.Oct 18 2015, 3:05 AM

changed according to Elena's comments

This revision was automatically updated to reflect the committed changes.