You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the codebase of the Polygeist project, there's a discrepancy between the function atan and atanh. It appears that math::AtanhOp is the suitable counterpart for atanh. However, the current implementation utilizes math.atan, which seems incorrect.
Furthermore, I've observed that Polygeist lacks support for several math operations, for example, math::TanhOP. I would like to implement support for new operations once I understand how the transformation works.
I've provided a sample code snippet below along with the generated MLIR code by Polygeist.
In the codebase of the Polygeist project, there's a discrepancy between the function
atan
andatanh
. It appears thatmath::AtanhOp
is the suitable counterpart foratanh
. However, the current implementation utilizesmath.atan
, which seems incorrect.Polygeist/tools/cgeist/Lib/CGCall.cc
Lines 979 to 986 in 8f8ae72
However,
math::AtanhOp
is introduced in a later commit of llvm-project, which is not available for the current submodule version.llvm/llvm-project@b8dca4f
Furthermore, I've observed that Polygeist lacks support for several math operations, for example,
math::TanhOP
. I would like to implement support for new operations once I understand how the transformation works.I've provided a sample code snippet below along with the generated MLIR code by Polygeist.
Current MLIR output by executing
cgeist foo.c -function=main -S
for the provided code snippet:The text was updated successfully, but these errors were encountered: