[PATCH] D75163: [analyzer][StreamChecker] Adding precall and refactoring.
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 4 06:11:31 PST 2020
Szelethus added a comment.
In D75163#1902816 <https://reviews.llvm.org/D75163#1902816>, @xazax.hun wrote:
> The name of the patch implies refactoring but some functional changes were also done. Is it possible to separate the functional changes into a separate patch? If it is not a big effort I would prefer that way.
It would indeed be nicer, but given that the patch is relatively small both in LOC and in scope, I think its okay. In the future, I would also prefer to separate functional and non-functional changes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75163/new/
https://reviews.llvm.org/D75163
More information about the cfe-commits
mailing list