[PATCH] D69950: Reapply "Fix crash on switch conditions of non-integer types in templates"

Elizabeth Andrews via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 11 12:58:19 PST 2019


eandrews added a comment.

The patch changes type dependency of field 'std::string s' and sets it based on field type (i.e. not type-dependent).  I do not know how clang-tidy works internally, but I assume checker used to skip the field earlier since it was set as dependent (based on containing class 'S'), hence not throwing the warning on any platform?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69950





More information about the cfe-commits mailing list