Skip to content

Commit

Permalink
Use correct node for value in VectorStore
Browse files Browse the repository at this point in the history
VectorStore properly looks to see if the value being
provided is a scalar and therefore nees to be splatted
to a vector, but then the actual store node generated
uses the original value (whoops!). Simple fix is to
use the (possibly splatted) updated node value.

Also added a missing TraceIL call that will help make it
easier to track what's going on in the traceIlGen output.

Signed-off-by: Mark Stoodley <mstoodle@ca.ibm.com>
  • Loading branch information
mstoodle committed Aug 20, 2018
1 parent 6c27ccd commit cf1ba22
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion compiler/ilgen/OMRIlBuilder.cpp
Expand Up @@ -660,7 +660,7 @@ OMR::IlBuilder::VectorStore(const char *varName, TR::IlValue *value)
TR::SymbolReference *symRef = lookupSymbol(varName);

TraceIL("IlBuilder[ %p ]::VectorStore %s %d gets %d\n", this, varName, symRef->getCPIndex(), value->getID());
storeNode(symRef, loadValue(value));
storeNode(symRef, valueNode);
}

void
Expand Down

0 comments on commit cf1ba22

Please sign in to comment.