[PATCH] D75682: [Analyzer][StreamChecker] Introduction of stream error handling.
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 27 05:24:16 PDT 2020
Szelethus added a comment.
In D75682#1940901 <https://reviews.llvm.org/D75682#1940901>, @balazske wrote:
> So what is missing or wrong in this change to be accepted?
This change is functional but is not testable. But the high level idea is great! I think we should maybe merge this with the followup, as you originally intended to.
Also, judging from the fact that how differently we imagined this to go (even though imo your idea is superior to mine), we should explain what our state splitting ideology is in the code.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75682/new/
https://reviews.llvm.org/D75682
More information about the cfe-commits
mailing list