[PATCH] D71524: [analyzer] Support tainted objects in GenericTaintChecker

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 24 09:26:16 PST 2020


Szelethus added a comment.

In D71524#1889566 <https://reviews.llvm.org/D71524#1889566>, @boga95 wrote:

> @steakhal's revision is on the top of this. Changing the order will only cause unnecessary work on both sides.


He recently rebased on top of master. I'm no fan of creating unnecessary work for either of you, and it might just be the case that my worry is greater then necessary. But for now, landing more patches with this infrastructure and having to painstakingly redo them all sounds like more work.

It'd be best to wait for @NoQ's feedback, after all, he did approve earlier patches. And again, sorry for not being available earlier.


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

https://reviews.llvm.org/D71524





More information about the cfe-commits mailing list