[all-commits] [llvm/llvm-project] 81aa66: Extract infrastructure to ignore intermediate expr...

Eduardo Caldas via All-commits all-commits at lists.llvm.org
Mon Sep 7 02:33:02 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 81aa66f65f504af18982baa078a5f3f7d2aa88fa
      https://github.com/llvm/llvm-project/commit/81aa66f65f504af18982baa078a5f3f7d2aa88fa
  Author: Eduardo Caldas <ecaldas at google.com>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

  Changed paths:
    A clang/include/clang/AST/IgnoreExpr.h
    M clang/lib/AST/CMakeLists.txt
    M clang/lib/AST/Expr.cpp
    A clang/lib/AST/IgnoreExpr.cpp

  Log Message:
  -----------
  Extract infrastructure to ignore intermediate expressions into `clang/AST/IgnoreExpr.h`

Rationale:
This allows users to use `IgnoreExprNodes` and `Ignore*SingleStep` outside of
`clang/AST/Expr.cpp`.

Minor:
Rename `IgnoreImp...SingleStep`  into `IgnoreImplicit...SingleStep`.

Differential Revision: https://reviews.llvm.org/D86778


  Commit: 1a7a2cd7474e6d321120ffe7ca9c52163eb228f0
      https://github.com/llvm/llvm-project/commit/1a7a2cd7474e6d321120ffe7ca9c52163eb228f0
  Author: Eduardo Caldas <ecaldas at google.com>
  Date:   2020-09-07 (Mon, 07 Sep 2020)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/UseAutoCheck.cpp
    M clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp
    M clang/include/clang/AST/Expr.h
    M clang/lib/AST/Expr.cpp
    M clang/lib/CodeGen/CGExprCXX.cpp
    M clang/lib/Sema/SemaExpr.cpp
    M clang/lib/StaticAnalyzer/Core/CallEvent.cpp

  Log Message:
  -----------
  [Ignore Expressions][NFC] Refactor to better use `IgnoreExpr.h` and nits

This change groups
* Rename: `ignoreParenBaseCasts` -> `IgnoreParenBaseCasts` for uniformity
* Rename: `IgnoreConversionOperator` -> `IgnoreConversionOperatorSingleStep` for uniformity
* Inline `IgnoreNoopCastsSingleStep` into a lambda inside `IgnoreNoopCasts`
* Refactor `IgnoreUnlessSpelledInSource` to make adequate use of `IgnoreExprNodes`

Differential Revision: https://reviews.llvm.org/D86880


Compare: https://github.com/llvm/llvm-project/compare/8d64df9f1390...1a7a2cd7474e


More information about the All-commits mailing list