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

Anna Zaks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 22 10:28:22 PDT 2017


zaks.anna added a comment.

This just supports the statement that this particular check should not go under unix. I understand that it will be inconsistent with the name of the malloc checker, which we probably should not change as people might be relying on the package names. I think it's better to have inconsistency than having checks applicable to windows in a package named portability.unix. If there will be checks that need to be in portability and only used for unix, we could create that sub-package later on.


https://reviews.llvm.org/D34102





More information about the cfe-commits mailing list