[PLACEHOLDER] Hoisted temporary zero initialization bug

[weiya711]

I think there is a bug with the check here:

taco/src/lower/lowerer_impl_imperative.cpp

Line 2215 in ee8c22a

if (util::contains(needCompute, temporary) && !isScalar(temporary.getType()) && !accelerateDenseWorkSpace) {

It should be if (temporaryHoisted && util::contains(needCompute, temporary) && !isScalar(temporary.getType()) && !accelerateDenseWorkSpace)

since the code initialization is handled in codeToInitializeTemporary() for the if (!temporaryHoisted) case here

@stephenchouca can you confirm this for me since you made some of these changes too?
Placeholder for @weiya711 (myself) to fix later

[weiya711]

I need to come up with an example that shows this bug

[GokuMizuno]

Since accelerateDenseWorkSpace is a bool, may I suggest placing it first in the if statement, to allow for short circuiting.