Skip to content

Feature/implement pointwise ops#289

Closed
Jagggged wants to merge 11 commits into
PSAL-POSTECH:masterfrom
Jagggged:feature/implement-pointwise-ops
Closed

Feature/implement pointwise ops#289
Jagggged wants to merge 11 commits into
PSAL-POSTECH:masterfrom
Jagggged:feature/implement-pointwise-ops

Conversation

@Jagggged

@Jagggged Jagggged commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@Jagggged Jagggged closed this Jul 8, 2026
@Jagggged Jagggged reopened this Jul 8, 2026
@YWHyuk

YWHyuk commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Thank you for your hard work.
Could you add this test into CI?

YWHyuk added 2 commits July 8, 2026 22:17
The transcendental ops (log, atan, atanh, cosh, sinh, acos, asin, acosh,
asinh, tanh) lower to float-only instructions. Promote non-float inputs
(integers) to f32 to run them while leaving native float widths (f16/f64)
untouched, so the ops stay dtype-preserving for floats. Also drop a
leftover isnan debug print.
Register the new pointwise op test in the explicit allowlist so it gates
PRs like the other tests.
@YWHyuk YWHyuk force-pushed the feature/implement-pointwise-ops branch from 626d425 to dc8fc23 Compare July 8, 2026 13:18
Bump the third-party pins to the releases that ship the new pointwise
support: spike v1.0.6 (torchsim_vlog/vatan) and gem5 v1.0.2 (CustomVlog/
Vatan opclasses plus vsin/vcos/vlog/vatan decode), so CI and the base
image pick them up.
@YWHyuk YWHyuk force-pushed the feature/implement-pointwise-ops branch from dc8fc23 to dd2b334 Compare July 8, 2026 13:30
@YWHyuk

YWHyuk commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

이 작업은 C++ trace 파이프라인(feature/togsim-cpp-trace) 위로 재편되어 #291 로 옮겨졌습니다.

trace 브랜치에서는 VCIX lowering이 C++ mlir-opt 패스가 아니라 in-process Python 패스(passes/lower_to_vcix.py)라서, 이 master 기반 PR을 그대로 rebase하면 컴파일 파이프라인(extension_codecache.py)에서 충돌합니다. 그래서 아키텍처에 맞춰 다시 구성했습니다:

  • pointwise 연산 lowering(mlir_ops.py)과 test_pointwise.py 그대로 이관 (원저자 커밋 authorship 유지)
  • llvm-project#10의 C++ log/atan VCIX 패턴은 Python 패스로 마이그레이션
  • extension_codecache.py의 tls_mode 변경은 trace에 해당 C++ 호출이 없어 제외

후속은 #291 에서 진행합니다. 닫습니다.

@YWHyuk YWHyuk closed this Jul 9, 2026
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.

2 participants