[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