[PATCH] D150370: Introduce StructuredData

[Sebastian Neubauer via Phabricator via llvm-commits]

Flakebi added a comment.

Why does the symbol map need to be global, shouldn’t it by per LLVMContext (which would also get rid of the locking)?

Needs tests.



================
Comment at: llvm/docs/LangRef.rst:3519-3523
+    sdata       ::= '{' (sdata_field ',')* sdata_field? '}'
+    sdata_field ::= label sdata_value
+    sdata_value ::= 'type' type
+                ::= integer
+                ::= 'true' | 'false'
----------------
Can constants follow the way metadata (and most other things in LLVM) is encoded and prefix the type?
I.e. `i32 <integer>` and `i1 true/false`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150370/new/

https://reviews.llvm.org/D150370