[PATCH] D77341: [DomTree] Replace ChildrenGetter with GraphTraits over GraphDiff.

[Uday Bondhugula via Phabricator via llvm-commits]

bondhugula added a comment.

@asbirlea This has broken the MLIR build. Could you please build with `-DLLVM_ENABLE_PROJECTS=mlir`?

  FAILED: tools/mlir/lib/Analysis/CMakeFiles/MLIRAnalysis.dir/Dominance.cpp.o 
  /usr/lib64/ccache/clang++  -DGTEST_HAS_RTTI=0 -DMLIR_CUDA_CONVERSIONS_ENABLED=1 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Itools/mlir/lib/Analysis -I/home/uday/llvm-project/mlir/lib/Analysis -I/usr/include/libxml2 -Iinclude -I/home/uday/llvm-project/llvm/include -I/home/uday/llvm-project/mlir/include -Itools/mlir/include -fPIC -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wstring-conversion -fdiagnostics-color -ffunction-sections -fdata-sections -O2     -fno-exceptions -fno-rtti -UNDEBUG -std=c++14 -MD -MT tools/mlir/lib/Analysis/CMakeFiles/MLIRAnalysis.dir/Dominance.cpp.o -MF tools/mlir/lib/Analysis/CMakeFiles/MLIRAnalysis.dir/Dominance.cpp.o.d -o tools/mlir/lib/Analysis/CMakeFiles/MLIRAnalysis.dir/Dominance.cpp.o -c /home/uday/llvm-project/mlir/lib/Analysis/Dominance.cpp
  In file included from /home/uday/llvm-project/mlir/lib/Analysis/Dominance.cpp:14:
  In file included from /home/uday/llvm-project/mlir/include/mlir/Analysis/Dominance.h:12:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/RegionGraphTraits.h:18:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/Region.h:16:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/Block.h:16:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/BlockSupport.h:16:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/Value.h:16:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/Types.h:12:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/TypeSupport.h:17:
  In file included from /home/uday/llvm-project/mlir/include/mlir/IR/StorageUniquerSupport.h:17:
  In file included from /home/uday/llvm-project/mlir/include/mlir/Support/STLExtras.h:18:
  In file included from /home/uday/llvm-project/llvm/include/llvm/ADT/STLExtras.h:20:
  /home/uday/llvm-project/llvm/include/llvm/ADT/iterator.h:366:17: error: indirection requires pointer operand ('const mlir::PredecessorIterator' invalid)
      NR.second = *this->I;


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77341/new/

https://reviews.llvm.org/D77341