[llvm-commits] [llvm] r97056 - /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp

[Douglas Gregor]

On Feb 24, 2010, at 12:59 PM, David Greene wrote:

> Author: greened
> Date: Wed Feb 24 14:59:49 2010
> New Revision: 97056
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=97056&view=rev
> Log:
> 
> Use a SmallPtrSet as suggested by Chris.

This seems to be breaking all of the buildbots, e.g.,

	http://google1.osuosl.org:8011/builders/llvm-i686-linux/builds/7815/steps/compile/logs/stdio

Did you try to build before committing?

	- Doug

> Modified:
>    llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
> 
> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp?rev=97056&r1=97055&r2=97056&view=diff
> ==============================================================================
> --- llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp (original)
> +++ llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp Wed Feb 24 14:59:49 2010
> @@ -6344,8 +6344,8 @@
> }
> 
> static void checkForCyclesHelper(const SDNode *N,
> -                                 std::set<const SDNode *> &visited,
> -                                 std::set<const SDNode *> &checked) {
> +                                 SmallPtrSet<const SDNode *, 32> &visited,
> +                                 SmallPtrSet<const SDNode *, 32> &checked) {
>   if (checked.find(N) != checked.end())
>     return;
> 
> @@ -6371,8 +6371,8 @@
> void llvm::checkForCycles(const llvm::SDNode *N) {
> #ifdef XDEBUG
>   assert(N && "Checking nonexistant SDNode");
> -  std::set<const SDNode *> visited;
> -  std::set<const SDNode *> checked;
> +  SmallPtrSet<const SDNode *, 32> visited;
> +  SmallPtrSet<const SDNode *, 32> checked;
>   checkForCyclesHelper(N, visited, checked);
> #endif
> }
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits