[PATCH] D33169: [X86] Adding vpopcntd and vpopcntq instructions

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:881
+    { X86::VPOPCNTDZrr,      X86::VPOPCNTDZrm,        TB_NO_REVERSE },
+    { X86::VPOPCNTQZrr,      X86::VPOPCNTQZrm,        TB_NO_REVERSE },
     { X86::VBROADCASTSSZr,   X86::VBROADCASTSSZm,     TB_NO_REVERSE },
----------------
Why TB_NO_REVERSE? This is typically only used for instructions where the mem size doesn't match the reg size to prevent out of bounds loads.


Repository:
  rL LLVM

https://reviews.llvm.org/D33169