[llvm] [Target][AMDGPU] Fix TSan error on AMDGPU Target. (PR #79529)

Jakub Kuderski via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 16:47:24 PST 2024


================
@@ -112,11 +112,15 @@ bool AMDGPUResourceUsageAnalysis::runOnModule(Module &M) {
 
   // By default, for code object v5 and later, track only the minimum scratch
   // size
+  uint32_t AssumedStackSizeForDynamicSizeObjects =
+      clAssumedStackSizeForDynamicSizeObjects.getDefault().getValue();
+  uint32_t AssumedStackSizeForExternalCall =
+      clAssumedStackSizeForExternalCall.getDefault().getValue();
----------------
kuhar wrote:

I don't understand why we need the `.getDefault()` -- wouldn't this make us ignore the value set in cli?

https://github.com/llvm/llvm-project/pull/79529


More information about the llvm-commits mailing list