[PATCH] D46425: Clear converters map after X86 Domain Reassignment to avoid crashes

Dimitry Andric via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 12 12:13:36 PDT 2018


dim updated this revision to Diff 146479.
dim added a comment.

Use `DeleteContainerSeconds` instead of rolling our own.


Repository:
  rL LLVM

https://reviews.llvm.org/D46425

Files:
  lib/Target/X86/X86DomainReassignment.cpp


Index: lib/Target/X86/X86DomainReassignment.cpp
===================================================================
--- lib/Target/X86/X86DomainReassignment.cpp
+++ lib/Target/X86/X86DomainReassignment.cpp
@@ -750,8 +750,7 @@
       Changed = true;
     }
 
-  for (auto I : Converters)
-    delete I.second;
+  DeleteContainerSeconds(Converters);
 
   DEBUG(dbgs() << "***** Machine Function after Domain Reassignment *****\n");
   DEBUG(MF.print(dbgs()));


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46425.146479.patch
Type: text/x-patch
Size: 458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180512/e0825f78/attachment.bin>


More information about the llvm-commits mailing list