This is an archive of the discontinued LLVM Phabricator instance.

GlobalISel: Translate ConstantAggregateZero vectors
ClosedPublic

Authored by volkan on Feb 22 2017, 9:00 AM.

Diff Detail

Event Timeline

volkan created this revision.Feb 22 2017, 9:00 AM
volkan updated this revision to Diff 91055.Mar 8 2017, 11:02 AM

Use G_MERGE_VALUES to represent vectors.

qcolombet accepted this revision.Mar 10 2017, 10:00 AM

LGTM

lib/CodeGen/GlobalISel/IRTranslator.cpp
984

I would just return false.
The caller should check the fallback/abort status.

I know surrounding code does that, but we should fix that as well. I'll have a try.

test/CodeGen/AArch64/GlobalISel/arm64-irtranslator.ll
1206

Add a non-power-of-2 test case

This revision is now accepted and ready to land.Mar 10 2017, 10:00 AM
volkan updated this revision to Diff 91391.Mar 10 2017, 12:00 PM
  • Added a non-power-of-2 test.
  • Removed GlobalISelAbortEnabled check.
volkan marked 2 inline comments as done.Mar 10 2017, 12:00 PM
volkan closed this revision.Mar 10 2017, 1:35 PM
qcolombet added inline comments.Mar 10 2017, 4:42 PM
lib/CodeGen/GlobalISel/IRTranslator.cpp
984

That's r297535