[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