arcana-lab / memoir

A case for representing data collections and objects in the LLVM IR
MIT License
11 stars 2 forks source link

TensorSummary Mistaken #28

Closed peteryongzhong closed 1 year ago

peteryongzhong commented 1 year ago

in commit 326f3e248b1d2f636b6bce1ac8475e9f21aa2024 , against https://github.com/arcana-lab/object-ir/blob/326f3e248b1d2f636b6bce1ac8475e9f21aa2024/tests/unit/test_tensor_really_simple/main.cpp .

I know this is against the old branch, but i couldn't get the new one (changed access api) to compile. I suspect the same issue would persist.

The printed number of dimensions and the size value is incorrect.

peteryongzhong commented 1 year ago
number of dimensions : 3
the value of the first index :   %4 = call %"struct.memoir::Object"* (%"struct.memoir::Type"*, i64, ...) @allocateTensor(%"struct.memoir::Type"* %3, i64 2, i32 100, i32 200) #6
peteryongzhong commented 1 year ago

No longer relevant