This is an archive of the discontinued LLVM Phabricator instance.

[X86][BF16] Do not scalarize masked load for BF16 when we have AVX512_BF16
ClosedPublic

Authored by pengfei on Jul 21 2023, 6:45 AM.

Diff Detail

Event Timeline

pengfei created this revision.Jul 21 2023, 6:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 21 2023, 6:45 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
pengfei requested review of this revision.Jul 21 2023, 6:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 21 2023, 6:45 AM
RKSimon accepted this revision.Jul 21 2023, 6:51 AM

LGTM

This revision is now accepted and ready to land.Jul 21 2023, 6:51 AM
This revision was landed with ongoing or failed builds.Jul 21 2023, 8:19 AM
This revision was automatically updated to reflect the committed changes.
pengfei reopened this revision.Jul 21 2023, 8:30 AM
This revision is now accepted and ready to land.Jul 21 2023, 8:30 AM
pengfei updated this revision to Diff 543157.Jul 22 2023, 3:19 AM

Change predicate to hasBF16. We don't have legal vector types before AVX512BF16.

pengfei retitled this revision from [X86][BF16] Do not scalarize masked load for BF16 when we have BWI to [X86][BF16] Do not scalarize masked load for BF16 when we have AVX512_BF16.Jul 22 2023, 3:20 AM
This revision was landed with ongoing or failed builds.Jul 22 2023, 3:21 AM
This revision was automatically updated to reflect the committed changes.