[PATCH] D52446: Remove Found.clear() to silent bugprone-use-after-move after rC342925

[Fangrui Song via Phabricator via cfe-commits]

MaskRay created this revision.
MaskRay added a reviewer: rsmith.
Herald added a subscriber: cfe-commits.
MaskRay added a reviewer: clang.

Repository:
  rC Clang

https://reviews.llvm.org/D52446

Files:
  lib/Sema/SemaStmt.cpp


Index: lib/Sema/SemaStmt.cpp
===================================================================
--- lib/Sema/SemaStmt.cpp
+++ lib/Sema/SemaStmt.cpp
@@ -2223,7 +2223,6 @@
           llvm::function_ref<Sema::ForRangeStatus()> BuildFound,
           llvm::function_ref<Sema::ForRangeStatus()> BuildNotFound) {
         LookupResult OldFound = std::move(Found);
-        Found.clear();
 
         if (Sema::ForRangeStatus Result = BuildNotFound())
           return Result;



Index: lib/Sema/SemaStmt.cpp
===================================================================
--- lib/Sema/SemaStmt.cpp
+++ lib/Sema/SemaStmt.cpp
@@ -2223,7 +2223,6 @@
           llvm::function_ref<Sema::ForRangeStatus()> BuildFound,
           llvm::function_ref<Sema::ForRangeStatus()> BuildNotFound) {
         LookupResult OldFound = std::move(Found);
-        Found.clear();
 
         if (Sema::ForRangeStatus Result = BuildNotFound())
           return Result;

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