Skip to content

Conversation

@j2kun
Copy link
Collaborator

@j2kun j2kun commented Sep 2, 2025

This should fix #885 (comment) but it needs way more testing

Tests run:

  • bazel run with yosys optimizer: bazel run //tools:heir-opt -- --yosys-optimizer $PWD/tests/Transforms/yosys_optimizer/arith_ops.mlir
  • bazel test a test target (for detecting runfiles dir) bazel test tests/Transforms/new_layout_propagation:add_one.mlir.test
  • Running from bazel-bin: bazel-bin/tools/heir-opt --yosys-optimizer --canonicalize --cse --secret-forget-secrets $PWD/tests/Transforms/yosys_optimizer/arith_ops.mlir
  • Frontend tests via bazel (bazel test frontend:e2e_test)

I still want to test:

  • What happens when I install from the night release (in which case abc and the techmap files are in the same directory as the binary)
  • Frontend tests when run from pip install -e .
  • Frontend tests when run from pip install .
  • Frontend tests when run from pip installing a pre-built wheel.

#vc4a

@j2kun j2kun marked this pull request as draft September 2, 2025 22:44
@j2kun j2kun force-pushed the sh_binary_wrapper branch from 1bf5955 to fd7692b Compare September 3, 2025 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot find abc binary when running heir-opt as a standalone binary

1 participant