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

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 24 18:09:58 PDT 2018


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;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52446.166795.patch
Type: text/x-patch
Size: 471 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180925/e7b40dbf/attachment.bin>


More information about the cfe-commits mailing list