Skip to content

Conversation

@cbalint13
Copy link
Contributor

@cbalint13 cbalint13 commented May 12, 2025

Tiny fixes for the refactored FFI backend & LLVM math macros.

  • FFI related compilation error:
# make -j8
{...}
In file included from tvm-0.20-build/tvm/ffi/include/tvm/ffi/container/array.h:29:
In file included from tvm-0.20-build/tvm/ffi/include/tvm/ffi/any.h:27:
tvm-0.20-build/tvm/ffi/include/tvm/ffi/string.h:214:10: error: field has incomplete type 'String'
  214 |   String(nullptr_t) = delete;  // NOLINT(*)
{...}
# rpm -q clang
clang-20.1.2-3.fc42.riscv64
intrin_rule_llvm.cc(185,48): error C2065: 'M_PI': undeclared identifier

@cbalint13 cbalint13 changed the title [FFI] Fix compilation errors with clang20 [FFI][LLVM] Fix compilation errors with clang20 May 12, 2025
@cbalint13 cbalint13 force-pushed the ffi-string branch 2 times, most recently from 204f07f to e3fe513 Compare May 12, 2025 10:39
@cbalint13 cbalint13 marked this pull request as draft May 12, 2025 10:40
@cbalint13 cbalint13 marked this pull request as ready for review May 12, 2025 12:45
@cbalint13
Copy link
Contributor Author

cbalint13 commented May 12, 2025

cc @Hzfengsy @Lunderberg @mshr-h
cc @tqchen

@mshr-h mshr-h merged commit ae053e6 into apache:main May 12, 2025
14 checks passed
ShiboXing pushed a commit to ShiboXing/tvm that referenced this pull request Aug 10, 2025
tqchen pushed a commit to tqchen/tvm that referenced this pull request Sep 13, 2025
tqchen pushed a commit to tqchen/tvm that referenced this pull request Sep 13, 2025
tqchen pushed a commit to tqchen/tvm that referenced this pull request Sep 13, 2025
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