[all-commits] [llvm/llvm-project] 77af8b: [mlir]Moves the StateStack to IR folder from Suppo...

[Lance Wang via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 77af8bff97a0b20dac9ff9a95385d036da7d8ba5
      https://github.com/llvm/llvm-project/commit/77af8bff97a0b20dac9ff9a95385d036da7d8ba5
  Author: Lance Wang <lancewang at google.com>
  Date:   2025-06-25 (Wed, 25 Jun 2025)

  Changed paths:
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    A mlir/include/mlir/IR/StateStack.h
    R mlir/include/mlir/Support/StateStack.h
    M mlir/include/mlir/Target/LLVMIR/ModuleTranslation.h
    M mlir/lib/IR/CMakeLists.txt
    A mlir/lib/IR/StateStack.cpp
    M mlir/lib/Support/CMakeLists.txt
    R mlir/lib/Support/StateStack.cpp
    M utils/bazel/llvm-project-overlay/mlir/BUILD.bazel

  Log Message:
  -----------
  [mlir]Moves the StateStack to IR folder from Support folder. (#145598)

[MLIR] Fix circular dependency introduced in In
https://github.com/llvm/llvm-project/pull/144897. This PR is to break
the dependency. by moving StateStack to IR folder

This commit resolves a circular dependency issue between mlir/Support
and mlir/IR:

- Move StateStack.h and StateStack.cpp from Support to IR folder
- Update CMakeLists.txt files to reflect the new locations
- Update Bazel BUILD file to maintain correct dependencies
- Update includes in affected files (flang, Target/LLVMIR)

The circular dependency was caused by StateStack.h depending on
IR/Visitors.h
while other IR files depended on Support. Moving StateStack to IR
eliminates
this cycle while maintaining proper separation of concerns.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications