[PATCH] D49844: [AST] Add a isActuallyImplicitCast() helper to the CastExpr class.
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 26 13:24:05 PDT 2018
rsmith requested changes to this revision.
rsmith added a comment.
This revision now requires changes to proceed.
What I requested was that either we make `CastExpr::isPartOfExplicitCast()` return `true` for `CastExpr`s that are not `ImplicitCastExpr`s, or that we move `isPartOfExplicitCast` do the `ImplicitCastExpr` derived class. We don't need to do both, and since we're doing the latter in https://reviews.llvm.org/D49838, we don't need to do anything else. (Essentially, my request was to fix the bug that `CastExpr::isPartOfExplicitCast()` incorrectly returned `false` when called on an explicit cast expression, and I don't mind which way we fix it.)
Repository:
rC Clang
https://reviews.llvm.org/D49844
More information about the cfe-commits
mailing list