These 5 Power-specific test cases fail with VSX support turned on. Originally they failed because fastisel does not support VSX registers and when one was encountered some "unexpected register class" assertions were triggered. Now they fail because fastisel returns that it could not handle things (see http://reviews.llvm.org/D5362) and they specify the -fast-isel-abort option. All this patch does is to specify -mattr=-vsx as a new option. Eventually we will want to remove this when fastisel fully supports VSX.
Diff Detail
Diff Detail
Event Timeline
Comment Actions
typo: "-fast-isel-abourt" in all the tests. Otherwise this is fine from my standpoint.
Comment Actions
These changes are harmless and will help keep the failure list down as we work on enabling VSX. As discussed in http://reviews.llvm.org/D5362, true VSX support for FastISel is planned, at which time VSX-specific tests (or variants of these tests) will be added. The patch LGTM -- I'll plan to commit on Bill Seurer's behalf tomorrow night if nobody objects.
Comment Actions
I committed this as r220172 today. I took the liberty of reflowing the comments because they were wider than 80 columns -- very naughty! /fingerwag