[PATCH] D49413: [x86/SLH] Flesh out the data-invariant instruction table a bit based on feedback from Craig.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 17 08:20:37 PDT 2018


chandlerc added a comment.

All done.



================
Comment at: llvm/lib/Target/X86/X86SpeculativeLoadHardening.cpp:963
   // instructions to exhibit that behavior.
   case X86::MULX32rr:
   case X86::MULX64rr:
----------------
craig.topper wrote:
> These are double width multiplies too. Should we drop them?
I mean, I *think* they're safe, but I'm very happy to go minimal for now.


Repository:
  rL LLVM

https://reviews.llvm.org/D49413





More information about the llvm-commits mailing list