This is an archive of the discontinued LLVM Phabricator instance.

[X86][PREFETCH]: Adding full coverage of MC encoding for the PREFETCH isa sets.<NFC>
ClosedPublic

Authored by gadi.haber on Dec 13 2017, 2:16 AM.

Details

Summary

NFC.
Adding MC regressions tests to cover the PREFETCH isa sets for both 32 and 64 bit.
This patch is part of a larger task to cover MC encoding of all X86 ISA Sets started in revision: https://reviews.llvm.org/D39952

Diff Detail

Repository
rL LLVM

Event Timeline

gadi.haber created this revision.Dec 13 2017, 2:16 AM

What about PREFETCHNTA/PREFETCH0/PREFETCH1/PREFETCH2 ?

PREFETCHNTA mem8 0F 18 /0 Move data closer to the processor using the NTA reference.
PREFETCHT0 mem8 0F 18 /1 Move data closer to the processor using the T0 reference.
PREFETCHT1 mem8 0F 18 /2 Move data closer to the processor using the T1 reference.
PREFETCHT2 mem8 0F 18 /3 Move data closer to the processor using the T2 reference.

Added missing tests for the prefetcht0/1/2 and prefetchnta instructions

I was going to tell you that prefetchwt1 is missing, but then I realized its only half implemented in llvm. We have a feature flag and no instruction definition.

prefetchwt1 is supported now. Can you add it to this test?

Sorry for delayed response. Just returned from vacation.
Added the prefetchwt1 testing.

This revision is now accepted and ready to land.Dec 27 2017, 11:49 AM
This revision was automatically updated to reflect the committed changes.