[clang] [llvm] [SPIRV][RFC] Rework / extend support for memory scopes (PR #106429)

Vyacheslav Levytskyy via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 16 05:33:17 PDT 2024


VyacheslavLevytskyy wrote:

> > @AlexVlx I don't see much objections against #108528 on a conceptual level, so what do you think about merging it into this PR in a way that I commented above, by changing `getMemScope()` and moving `getOrInsertSyncScopeID()` into its static vars initialization?
> 
> At a glance it seems fine, thank you for working through this, but since I've been away for a couple of days let me page things back in first:)

Sure, it's just that some parts of the discussion may be irrelevant if you'd choose the https://github.com/llvm/llvm-project/pull/108528 approach :)

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


More information about the cfe-commits mailing list