[PATCH] D144154: [X86]Use Class to refactor ArithMetic td file in X86

[Wang, Xin via Phabricator via llvm-commits]

XinWang10 added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrArithmetic.td:478
+//  UnaryOpM_mul - Instructions like "mul [mem]"
+class UnaryOpM_mul<bits<8> opcode, Format f, string mnemonic, X86TypeInfo info,
+              list<dag> pattern>
----------------
skan wrote:
> Please make sure that a class is not defined when it is only used at one place, otherwise it's too overdesign.
This is designed to be consistent with other class I wrote, it is unnecessary but make sense to me? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144154