[PATCH] D48820: [llvm-exegesis] ExegisX86Target::setRegToConstant() should depend on the subtarget features.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 2 09:54:31 PDT 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one minor



================
Comment at: unittests/tools/llvm-exegesis/X86/TargetTest.cpp:93
+  EXPECT_EQ(Insts[6].getOpcode(), llvm::X86::ADD64ri8);
+}
+
----------------
Please can you YMM and ZMM tests as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D48820





More information about the llvm-commits mailing list