================
@@ -505,6 +800,12 @@ TEST_F(JSONFormatTUSummaryTest, 
NamespaceElementInvalidKind) {
         }
       }
     ],
+    "linkage_table": [
+      {
+        "id": 0,
+        "linkage": { "type": "external" }
+      }
+    ],
----------------
steakhal wrote:

I think now I understand what I dislike with this representation.
There is a lot of syntactic noise with this, while structuring it like this 
would make it more compact:
```
    "linkage_table": {
      "none": [1,2,31],
      "internal": [],
      "external": [0]
    },
```
We agree to not optimize the JSON layout but if we don't assume more linkage 
types, I think it's a sensible step to do.

https://github.com/llvm/llvm-project/pull/182961
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to