[PATCH] D34102: [analyzer] Add portability package for the checkers.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 03:20:11 PDT 2017


NoQ added a comment.

In https://reviews.llvm.org/D34102#788138, @zaks.anna wrote:

> I think it's better to have inconsistency than having checks applicable to windows in a package named portability.unix.




In https://reviews.llvm.org/D34102#788138, @zaks.anna wrote:

> If there will be checks that need to be in portability and only used for unix, we could create that sub-package later on.


Sounds good to me, will commit then. Still, more reasons to have aliases or tags.


https://reviews.llvm.org/D34102





More information about the cfe-commits mailing list