Lowering complex.tanh to standard dialects including math, arith.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/Conversion/ComplexToStandard/ComplexToStandard.cpp | ||
---|---|---|
732 | please, add a comment about what you're actually computing here and, if possible, where did you take it from. Here you are computing tanh(x + i * y) = (tanh(x) + i * tan(y)) / (1 + tanh(x) * tan(y)) | |
749 | Please, introduce TanOp for complex numbers that expands as sin/cos. Not necessarily in this patch. |
Comment Actions
@Harbormaster Thank you for the review. I'm going to prepare the patch for general tan operation for a complex number. Could you land this patch on behalf of me because I do not have permission to do so.
Comment Actions
@pifon2a Thank you for the review. I fixed the format issue and now all CI seems green. Could you push the patch into the repository because I don't have permission to do so.
please, add a comment about what you're actually computing here and, if possible, where did you take it from.
Here you are computing