[PATCH] D126224: Add DWARF string debug to clang release notes.

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 23 10:30:48 PDT 2022


rnk added a comment.

I am reminded of the perennial problem of "optional" protobuf fields that, when omitted, will cause production crashes.

Do you think it would be less disruptive to synthesize a name? I believe the string lives in a string pool, so naming all string literals `<stringliteral>` seems like it could be acceptable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126224



More information about the cfe-commits mailing list