[PATCH] D101115: [AMDGPU] Move code sinking before structurizer

[Piotr Sobczak via Phabricator via llvm-commits]

piotr added inline comments.


================
Comment at: llvm/test/CodeGen/AMDGPU/multilevel-break.ll:198
 ; GCN-NEXT:    ; =>This Inner Loop Header: Depth=1
-; GCN-NEXT:    buffer_load_dword v1, off, s[0:3], 0
+; GCN-NEXT:    buffer_load_dword v1, off, s[0:3], 0 glc
 ; GCN-NEXT:    s_waitcnt vmcnt(0)
----------------
arsenm wrote:
> arsenm wrote:
> > Did this drop the memory operand or something? This looks like a regression
> Oh, these loads are already volatile and should have had glc set. Maybe this test was last regenerated before glc was emitted for volatile loads?
Good spot - I get the glc generated even without my patch. I will pre-commit those glc changes separately so they do not pop up here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101115