[PATCH] D116779: [llvm-mca] [LSUnit] Proposal for declaring memory-barrier instructions explicitly rather than making conservative assumptions.

[Patrick Holland via Phabricator via llvm-commits]

holland11 updated this revision to Diff 398316.
holland11 added a comment.

@andreadb Thanks for the feedback / suggestions. They all made sense and I implemented each of them. Let me know if they look how you expected them to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116779

Files:
  llvm/docs/CommandGuide/llvm-mca.rst
  llvm/include/llvm/MCA/CustomBehaviour.h
  llvm/include/llvm/MCA/Instruction.h
  llvm/lib/MCA/HardwareUnits/LSUnit.cpp
  llvm/lib/Target/X86/CMakeLists.txt
  llvm/lib/Target/X86/MCA/CMakeLists.txt
  llvm/lib/Target/X86/MCA/X86CustomBehaviour.cpp
  llvm/lib/Target/X86/MCA/X86CustomBehaviour.h
  llvm/test/tools/llvm-mca/AArch64/Cortex/A55-load-store-noalias.s
  llvm/test/tools/llvm-mca/AMDGPU/gfx9-retireooo.s
  llvm/test/tools/llvm-mca/X86/Barcelona/store-throughput.s
  llvm/test/tools/llvm-mca/X86/BdVer2/load-store-throughput.s
  llvm/test/tools/llvm-mca/X86/BdVer2/pr37790.s
  llvm/test/tools/llvm-mca/X86/BdVer2/store-throughput.s
  llvm/test/tools/llvm-mca/X86/BtVer2/pr37790.s
  llvm/test/tools/llvm-mca/X86/BtVer2/stmxcsr-ldmxcsr.s
  llvm/test/tools/llvm-mca/X86/Haswell/reserved-resources.s
  llvm/test/tools/llvm-mca/X86/Haswell/stmxcsr-ldmxcsr.s
  llvm/test/tools/llvm-mca/X86/barrier_output.s
  llvm/tools/llvm-mca/Views/InstructionInfoView.cpp
  llvm/tools/llvm-mca/Views/InstructionInfoView.h
  llvm/tools/llvm-mca/llvm-mca.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116779.398316.patch
Type: text/x-patch
Size: 69157 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220108/06386dd1/attachment.bin>