[PATCH] D21723: [RFC] Enhance synchscope representation

[Konstantin Zhuravlyov via Phabricator via llvm-commits]

kzhuravl added inline comments.


================
Comment at: lib/IR/LLVMContextImpl.cpp:218
 
+SyncScope::ID LLVMContextImpl::getSyncScopeID(StringRef SSN) const {
+  auto SSI = SSC.find(SSN);
----------------
pcc wrote:
> Unused function
I was planning to start using it in the follow up patch, which implements AMDGPU's memory model. Would that be ok to leave this function in this patch? Or should it be moved to the follow up patch?


https://reviews.llvm.org/D21723