[PATCH] D21723: [RFC] Enhance synchscope representation

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: include/llvm/Bitcode/LLVMBitCodes.h:381
+  /// First encoded value for target specific synchronization scope.
+  SYNCHSCOPE_FIRSTTARGETSPECIFIC = 2
 };
----------------
mehdi_amini wrote:
> 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?
I think I mentioned in the past using target specific strings instead of integer in textual and bitcode serialization, why was this ruled out? This was intended to addressed such issues (and make the textual IR more readable)


https://reviews.llvm.org/D21723