[PATCH] D83837: [LLVM] Fix handling of escaped closing brace `}` in `formatv` (under an option)

Sam McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 15 10:04:25 PDT 2020


sammccall added a comment.

Thanks. I do think the strict behavior you describe here is a little bit better, but it's really hard to completely get there (not least because format strings often aren't tested...).
Formalizing the existing behavior is better than the halfway state.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83837





More information about the llvm-commits mailing list