Skip to content

Conversation

@vmaksimo
Copy link
Contributor

This continues #3343 and reflects specification update, including extension renaming. Specification: intel/llvm#20009

@vmaksimo
Copy link
Contributor Author

About memory scope change - it is expected, cause the fix for correct scope translation was added only in llvm-20 release branch 630a90a#diff-ba003da77a80df1db76fb378a873a80e758591d72bf82a46fc90069b2a947990R1904
And for this and earlier release branches we use Device scope by default:

SPIRVValue *LLVMToSPIRVBase::transAtomicStore(StoreInst *ST,
SPIRVBasicBlock *BB) {
SmallVector<StringRef> SSIDs;
ST->getContext().getSyncScopeNames(SSIDs);
spv::Scope S;
// Fill unknown syncscope value to default Device scope.
if (!OCLStrMemScopeMap::find(SSIDs[ST->getSyncScopeID()].str(), &S)) {
S = ScopeDevice;
}

@vmaksimo vmaksimo merged commit 1acae9d into KhronosGroup:llvm_release_190 Nov 20, 2025
9 checks passed
@vmaksimo vmaksimo deleted the bp-19-SPV_INTEL_16bit_atomics branch November 20, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant