[PATCH] D21723: [RFC] Enhance synchscope representation

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

We need an entry in the bitcode compatibility test as well.



================
Comment at: include/llvm/Bitcode/LLVMBitCodes.h:381
+  /// First encoded value for target specific synchronization scope.
+  SYNCHSCOPE_FIRSTTARGETSPECIFIC = 2
 };
----------------
if the plan is to be able to add new scopes (cf previous question), how is it supposed to be translated on the encoding side?


================
Comment at: include/llvm/IR/Instructions.h:49
+  /// First target specific synchronization scope.
+  SynchronizationScopeFirstTargetSpecific = 2
 };
----------------
Is the plan to be able to add new scopes?


https://reviews.llvm.org/D21723