[PATCH] D44726: [X86] Add ReadAfterLds to some 3 src instructions

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 20 21:40:02 PDT 2018


craig.topper created this revision.
craig.topper added reviewers: RKSimon, atrick.

I think we're supposed to have a ReadAfterLd for each operand that's read after load?

Sometimes the operand comes after the memory operand so we need 5 ReadDefaults first.

I suspect we also need to do something for the mask operand for masked avx512 instructions? I'm not sure if the mask should be ReadAfterLd or not since it can mask faults. If it shouldn't be ReadAfterLd then we're probably wrong for zero masking instructions already.


https://reviews.llvm.org/D44726

Files:
  lib/Target/X86/X86InstrFMA.td
  lib/Target/X86/X86InstrSSE.td
  lib/Target/X86/X86InstrShiftRotate.td
  lib/Target/X86/X86InstrXOP.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44726.139250.patch
Type: text/x-patch
Size: 12799 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180321/eec562bd/attachment.bin>


More information about the llvm-commits mailing list