[PATCH] D46054: [TableGen] Add a general-purpose JSON backend.

Nicolai Hähnle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 30 06:36:06 PDT 2018


nhaehnle added a comment.

There are arguments both for !key + !name and for !name + !anonymous, although thinking about it for a minute or two I weakly prefer !name + !anonymous because it matches the representation in C++. It makes it easier for people to move between JSON and C++.

P.S.: No worries about the confusion with the other review :)


https://reviews.llvm.org/D46054





More information about the llvm-commits mailing list