[PATCH] D73095: [llvm-exegesis] Fix support for LEA64_32r.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 21 04:39:10 PST 2020


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

LGTM appart a few typos



================
Comment at: llvm/tools/llvm-exegesis/lib/X86/Target.cpp:276
+               BitVector &CandidateDestRegs) {
+          // We just select an destination register that aliases the base
+          // register.
----------------
`a destination`


================
Comment at: llvm/tools/llvm-exegesis/lib/X86/Target.cpp:330
+               BitVector &CandidateDestRegs) {
+          // Any destination register that is not used for adddressing is fine.
+          remove(CandidateDestRegs,
----------------
`addressing`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73095/new/

https://reviews.llvm.org/D73095





More information about the llvm-commits mailing list