Skip to content

Commit 4028b40

Browse files
committed
[PyCDE] Remove behavioral if-else-end construct
Python 3.13 broke this functionality. We were never happy with the syntax so rather than fixing it, I'm just removing it. If anyone was using it (we're not using it internally), reach out to me to discuss better ways of doing this.
1 parent 8cb12b5 commit 4028b40

File tree

4 files changed

+10
-292
lines changed

4 files changed

+10
-292
lines changed

.github/workflows/pycdePublish.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,11 @@ jobs:
3333
- cp312-manylinux_x86_64
3434
- cp313-manylinux_x86_64
3535
steps:
36+
- name: Setup Python
37+
uses: actions/setup-python@v5
38+
with:
39+
python-version: "3.12"
40+
3641
- name: Get CIRCT (no submodules)
3742
uses: actions/checkout@v4
3843
with:
@@ -107,6 +112,11 @@ jobs:
107112
- cp312-win_amd64
108113
- cp313-win_amd64
109114
steps:
115+
- name: Setup Python
116+
uses: actions/setup-python@v5
117+
with:
118+
python-version: "3.12"
119+
110120
# Since we don't use docker on Windows, we need to install the dependencies.
111121
- name: Build additional c++ deps
112122
shell: pwsh

frontends/PyCDE/src/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ declare_mlir_python_sources(PyCDESources
2626
pycde/dialects/hwarith.py
2727
pycde/support.py
2828
pycde/module.py
29-
pycde/behavioral.py
3029
pycde/constructs.py
3130
pycde/common.py
3231
pycde/system.py

frontends/PyCDE/src/pycde/behavioral.py

Lines changed: 0 additions & 164 deletions
This file was deleted.

frontends/PyCDE/test/test_behavioral.py

Lines changed: 0 additions & 127 deletions
This file was deleted.

0 commit comments

Comments
 (0)