[PATCH] D56303: Handle case/default statements when simplifying boolean expressions

Richard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 3 17:33:06 PST 2019


LegalizeAdulthood created this revision.
LegalizeAdulthood added a reviewer: cfe-commits.
LegalizeAdulthood added a project: clang-tools-extra.

Structure this similarly to the way replaceCompoundReturnWithCondition works.

- Add readability-simplify-boolean-expr test cases for case stmt
- Add readability-simplify-boolean-expr test cases for default stmt
- Add negative test cases

Fixes #26704


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D56303

Files:
  clang-tidy/readability/SimplifyBooleanExprCheck.cpp
  clang-tidy/readability/SimplifyBooleanExprCheck.h
  test/clang-tidy/readability-simplify-bool-expr-case.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56303.180183.patch
Type: text/x-patch
Size: 28585 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190104/e8340c5f/attachment-0001.bin>


More information about the cfe-commits mailing list