Repository navigation
Conversation
Introduce MacroStatement as a leaf IET statement and generate standalone macro invocations through CGen. Cover code generation, statement mapping, node discovery and transformations. Fixes devitocodes#2853.
mloubout
left a comment
There was a problem hiding this comment.
Thank you for the contribution.
However this doesn't really implement the node, there is plenty of macro through the compiler and if a node for Macro is added they should all be switched to it.
Currently this is an unused change that is only tested standalone, not as part of the compiler.
|
Updated in a1c1bb5. Added compiler-path tests, including compiling and running C/OpenMP Operators. Merged the latest Validation:
Could you take another look? Thanks!
|
Introduce MacroStatement as a leaf IET statement and generate standalone macro invocations through CGen. Cover code generation, statement mapping, node discovery and transformations.
Fixes #2853.
Validation on macOS arm64 with Python 3.14.4, SymPy 1.14.0 and LLVM/Clang OpenMP:
tests/test_iet.py tests/test_visitors.py: 58 passed.pytest -q -n 2 -m "not parallel and not decoupler" tests/: 3667 passed, 50 skipped, 4 xfailed in 857.63 seconds.MacroStatementcompiled withclang -Wall -Wextra -Werrorand ran successfully.GPU execution, MPI/decoupler tests and the full CI platform matrix were not run locally.