[PATCH] D135161: [clang][Lex] Fix a crash on malformed string literals
Corentin Jabot via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 6 02:46:24 PDT 2022
cor3ntin added a comment.
In D135161#3839510 <https://reviews.llvm.org/D135161#3839510>, @kadircet wrote:
> i'll reland the fix without the test case, as it's clearly fixing one of the codepaths that'll lead to a crash. it's only the test case that's crashing, because i don't think there are certain test cases that exposed literal parser to invalid/incomplete input and i am not able to reproduce any of the crashes i've seen in buildbots locally (even under asan/msan) and because the bots don't have stack traces for whatever reason, it's impossible to chase from my side. i've pinged buildbot owners in https://discourse.llvm.org/t/buildbots-missing-stack-traces/65753 to see the issue fixed.
Makes sense. The fix is correct and if it wasn't it would affect all bots equally (I highly doubt that code path is exercised anywhere but the lex test)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135161/new/
https://reviews.llvm.org/D135161
More information about the cfe-commits
mailing list