[PATCH] D64197: [HardwareLoops] NFC - move hardware loop checking code to isHardwareLoopProfitable()

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added a comment.

I tried adding a dependency on TransformUtils to Analysis/LLVMBuild.txt but that generated:

  CMake Error at CMakeLists.txt:711 (message):
    Unexpected failure executing llvm-build: llvm-build: fatal error: found
    cycle to u'Analysis' after following: u'AArch64CodeGen' (required library)
    -> u'Analysis' (required library) -> u'TransformUtils' (required library)
    -> Analysis

Does this mean that this change violates the dependency graph?    Assuming you can repro, with `-DBUILD_SHARED_LIBS=ON` then perhaps we can revert this while a solution is found?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64197