[PATCH] D99205: Add jump-threading optimization for deterministic finite automata

[Geoffrey Martin-Noble via Phabricator via llvm-commits]

GMNGeoffrey added a subscriber: bkramer.
GMNGeoffrey added a comment.

In D99205#2908261 <https://reviews.llvm.org/D99205#2908261>, @GMNGeoffrey wrote:

> This appears to create a layering violation. llvm/lib/Transforms/Scalar/DFAJumpThreading.cpp includes llvm/include/llvm/CodeGen/Passes.h and llvm/lib/CodeGen/HardwareLoops.cpp includes  llvm/include/llvm/Transforms/Scalar.h creating a cycle between LLVMCodeGen and LLVMScalarOpts
>
> @chandlerc as the owner of layering
>
> Is the include of llvm/CodeGen/Passes.h required? My build appears to succeed without it

Ah looks like @bkramer just fixed it with https://github.com/llvm/llvm-project/commit/05815c9f638c2a62e1ce9b28b26d74c7bea81f2e, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99205