[PATCH] D44093: [BUILTINS] structure pretty printer

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 12 07:12:15 PDT 2018

lebedev.ri added a comment.

BTW, as far as i can tell this still has zero test coverage (no new tests are being added).
I'd expect to see the tests for the actual output

- one struct per each type it is able to print
- probably some tests showing error handling,

and possibly the availability of the builtin is somehow tested, too?

  rC Clang


More information about the cfe-commits mailing list