Skip to content

taco: xorOp higher-order tensor code and Merge Lattice generation bug #9

Open
@weiya711

Description

@weiya711

Currently the taco/array_algebra branch generates incorrect xorOp lowering code for 2+ order tensors (found by @rohany)

The lowering code seems to only compare for the iteration algebra on the highest level of coordinates and does not check the case where input1_crd1 == input2_crd1 but input1_crd2 != input2_crd2.

Created a branch in taco/array_algebra_xor_fix that:

  1. adds in a new test called XorOpOrder2 in test/tests-lower.cpp to test the 2-dimensional C(i, j) = xorOp(A(i, j), B(i, j))
  2. Applies a patch from @rohany in taco/lower/lowerer-impl.cpp in lowerMergeCases(...) that should fix this bug

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions