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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 7 10:53:04 PDT 2018


craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86DomainReassignment.cpp:753
 
   for (auto I : Converters)
     delete I.second;
----------------
Use DeleteContainerSeconds from STLExtras.h, or better yet change Converts to hold std::unique_ptrs and then just call clear.


Repository:
  rL LLVM

https://reviews.llvm.org/D46425





More information about the llvm-commits mailing list