[PATCH] D96643: [AMDGPU] Limit memory scope for scratch and LDS

Madhur Amilkanthwar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 13 01:34:20 PST 2021


madhur13490 added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIMemoryLegalizer.cpp:144
+      if (Scope > SIAtomicScope::SINGLETHREAD)
+        this->Scope = SIAtomicScope::SINGLETHREAD;
+      break;
----------------
Why not?

```
this->scope = std::min(Scope, SIAtomicScope::SINGLETHREAD)
```








Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96643



More information about the llvm-commits mailing list