diff options
author | Nico Weber <thakis@chromium.org> | 2019-11-17 02:09:25 -0500 |
---|---|---|
committer | Nico Weber <thakis@chromium.org> | 2019-11-17 02:09:25 -0500 |
commit | c9276fbfdf0c7caf1576b2db562286d7cec7e53b (patch) | |
tree | b584cd30cae5f36372ed6fd48cb2cc4abec54dfa /clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp | |
parent | cc6b85390170725d2341416af2b2ab8ea6c25d83 (diff) | |
download | bcm5719-llvm-c9276fbfdf0c7caf1576b2db562286d7cec7e53b.tar.gz bcm5719-llvm-c9276fbfdf0c7caf1576b2db562286d7cec7e53b.zip |
Revert "[NFC] Refactor representation of materialized temporaries"
This reverts commit 08ea1ee2db5f9d6460fef1d79d0d1d1a5eb78982.
It broke ./ClangdTests/FindExplicitReferencesTest.All
on the bots, see comments on https://reviews.llvm.org/D69360
Diffstat (limited to 'clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp')
-rw-r--r-- | clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp b/clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp index a5206554091..e3b90d3b427 100644 --- a/clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp +++ b/clang-tools-extra/clang-tidy/performance/ImplicitConversionInLoopCheck.cpp @@ -78,7 +78,7 @@ void ImplicitConversionInLoopCheck::check( // iterator returns a value instead of a reference, and the loop variable // is a reference. This situation is fine (it probably produces the same // code at the end). - if (IsNonTrivialImplicitCast(Materialized->getSubExpr())) + if (IsNonTrivialImplicitCast(Materialized->getTemporary())) ReportAndFix(Result.Context, VD, OperatorCall); } |