[PATCH] D95312: [TableGen] [DetailedRecords] Print record name that is null string as ""

Paul C. Anagnostopoulos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 24 15:15:13 PST 2021


Paul-C-Anagnostopoulos added a comment.

Yes, of course, empty().

Are you asking whether there is a test for this change? There is no test at all for the DetailedRecord backend, just as there is no test for the default print backend. Do you think there should be tests for those two backends?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95312



More information about the llvm-commits mailing list