This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SVE] Added GPR64shifted and GPR64NoXZRshifted register classes.
ClosedPublic

Authored by sdesmalen on Apr 16 2018, 7:34 AM.

Diff Detail

Event Timeline

sdesmalen created this revision.Apr 16 2018, 7:34 AM
sdesmalen updated this revision to Diff 142775.Apr 17 2018, 7:49 AM

Minor clean-up of patch:

  • Removed template argument from 'printRegWithShiftExtend'
  • Added unused attribute to DecodeGPR64commonRegisterClass (will be removed in next patch, but avoids breaking buildbot).
  • Rephrased diagnostic to say 'xzr' instead of 'x31'.
sdesmalen edited the summary of this revision. (Show Details)Apr 19 2018, 12:26 AM
SjoerdMeijer accepted this revision.Apr 19 2018, 1:07 AM

Looks good to me, just one nit inlined.

lib/Target/AArch64/InstPrinter/AArch64InstPrinter.cpp
972

Nit: SignExtend and DoShift can be booleans?

989

Same here?

This revision is now accepted and ready to land.Apr 19 2018, 1:07 AM
sdesmalen added inline comments.Apr 19 2018, 6:42 AM
lib/Target/AArch64/InstPrinter/AArch64InstPrinter.cpp
972

Yes they can! I will change that before I commit, thanks for pointing out!

This revision was automatically updated to reflect the committed changes.