[PATCH] D91086: AMDGPU: Document why we use (non-volatile) BUFFER_WBINVL1 in graphics

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIMemoryLegalizer.cpp:904
+  // is NC_NV (non-coherent, non-volatile) on gfx8 (and maybe on gfx7, but not
+  // on gfx9), so a non-volatile wbinv is required.
   const unsigned InvalidateL1 = STM.isAmdPalOS() || STM.isMesa3DOS()
----------------
BUFFER_WBINVL1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91086