This is an archive of the discontinued LLVM Phabricator instance.

[PPC] Add Subtarget feature slower ISEL targets. NFC
Needs ReviewPublic

Authored by iteratee on Jan 11 2016, 2:02 PM.

Details

Reviewers
hfinkel
Summary

Power7 and Power8 have isel instructions that are slow compared to a
predicted branch. Document this as a subtarget feature.

Diff Detail

Event Timeline

iteratee updated this revision to Diff 44551.Jan 11 2016, 2:02 PM
iteratee retitled this revision from to [PPC] Add Subtarget feature slower ISEL targets. NFC.
iteratee updated this object.
iteratee added a reviewer: hfinkel.
iteratee added a subscriber: llvm-commits.
hfinkel edited edge metadata.Apr 26 2016, 6:48 PM

I think it would be best to make HasISEL a { Fast, Slow, No }-type enum. We can have separate feature strings for the llc command line, but otherwise it becomes difficult to make sure that the -misel -mno-isel options in Clang override things in the correct way. Please experiment with that -- that's the underlying design constraint.

lib/Target/PowerPC/PPC.td
90

If this line is longer that 80 characters, please unindent it as with other similar lines.