This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by craig.topper on Dec 15 2017, 4:14 AM.

Details

Summary

NFC.
Adding MC regressions tests to cover the XSAVE ISA sets.
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 15 2017, 4:14 AM
craig.topper added inline comments.Dec 15 2017, 8:30 AM
test/MC/X86/XSAVEC-32.s
21 ↗(On Diff #127097)

xsavec64 and friends shouldn't be allowed in a 32-bit test. The fact they are is a bug I'm going to fix now.

craig.topper commandeered this revision.Jan 12 2018, 9:23 AM
craig.topper edited reviewers, added: gadi.haber; removed: craig.topper.

Commandeering so I can remove the 64-bit instructions from the 32-bit tests.

Remove 64-bit instructions from 32-bit mode tests

This revision is now accepted and ready to land.Jan 15 2018, 11:43 AM
This revision was automatically updated to reflect the committed changes.