Szelethus accepted this revision. Szelethus added a comment. > "putting all such stuff into the `ProgramState` class clearly doesn't scale, > but we still didn't come up with anything better, so let's just put it in the > header and see if a better design suddenly emerges in the future after we do > it a few hundred times"
Now that we have a maintainer who will dedicate a lot of effort into taint analysis, that might just happen :) For the time being though, I agree, this is much cleaner. It would be great to land this as soon as you're comfortable with it, in order not to block @boga95's work. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59861/new/ https://reviews.llvm.org/D59861 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
