[llvm] r204431 - [Constant Hoisting] Replace the MapVector with a separate Map and Vector to keep track of constant candidates.

[Juergen Ributzka]

I could clear the map right after constant candidate collection is done, because it is only used there. Or I could make it a local map that never gets exposed to the other method.

On Mar 21, 2014, at 1:54 PM, David Blaikie <dblaikie at gmail.com> wrote:

> Would it be useful to invalidate the map and/or vector when this sort
> operation occurs? Perhaps emptying both the map and vector (simply
> std::move the vector into a local, sort it, use it, then leave the
> function with both map and vector empty - since leaving them non-empty
> is just a trap for someone later?)

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140325/88de9d7f/attachment.html>