This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Clean-up mayRaiseFPException bits
ClosedPublic

Authored by qiucf on Sep 15 2020, 8:54 PM.

Details

Summary

Instructions added mayRaiseFPException:

  • fre(s)/frsqrte(s)
  • fmadd(s)/fmsub(s)/fnmadd(s)/fnmsub(s)
  • xscmpoqp/xscmpuqp/xscmpeqdp/xscmpgedp/xscmpgtdp
  • xscvdphp/xscvhpdp/xvcvhpsp/xvcvsphp/xsrqpxp
  • xsmaxcdp/xsincdp/xsmaxjdp/xsminjdp

Instructions removed mayRaiseFPException:

  • xstdivdp/xvtdiv(d|s)p/xstsqrtdp/xvtsqrt(d|s)p
  • xsabsdp/xsnabsdp/xvabs(d|s)p/xvnabs(d|s)p
  • xsnegdp/xscpsgndp/xvneg(d|s)p/xvcpsgn(d|s)p
  • xvcvsxwdp/xvcvuxwdp
  • xscvdpspn/xscvspdpn

Diff Detail

Event Timeline

qiucf created this revision.Sep 15 2020, 8:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 15 2020, 8:54 PM
qiucf requested review of this revision.Sep 15 2020, 8:54 PM
qiucf edited the summary of this revision. (Show Details)Sep 15 2020, 11:24 PM
qiucf added reviewers: jsji, nemanjai, steven.zhang, Restricted Project, ZhangKang.
steven.zhang accepted this revision.Sep 26 2020, 5:13 PM

LGTM and we need some follow up patch to model the FPSCR and remove the hasSideEffects bit.

This revision is now accepted and ready to land.Sep 26 2020, 5:13 PM
This revision was automatically updated to reflect the committed changes.