[PATCH] D41329: [X86][CET]: Adding full coverage of MC encoding for the CET instructions.<NFC>

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 4 13:38:09 PST 2018


RKSimon added a comment.

In https://reviews.llvm.org/D41329#997307, @gadi.haber wrote:

> The MC tests for the CET ISA Set are fully covered in test/MC/X86/cet-encoding.s  added in https://reviews.llvm.org/D40223
>  I, therefore, abandon this review.


Don't we still need to add 32-bit coverage tests?


Repository:
  rL LLVM

https://reviews.llvm.org/D41329





More information about the llvm-commits mailing list