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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to