This is an archive of the discontinued LLVM Phabricator instance.

Partial support for Intel SHA Extensions (sha1rnds4)
AbandonedPublic

Authored by benlangmuir on Sep 11 2013, 2:03 PM.

Details

Reviewers
nadav
Summary

Add basic assembly/disassembly support for the first Intel SHA
instruction 'sha1rnds4'. Also includes feature flag, and test cases.

I've only included one instruction so that I can get early review as I have no
experience with the instruction patterns in tablegen.

Support for the remainin instructions will follow in a separate patch.

Diff Detail

Event Timeline

Comments inline.

benlangmuir updated this revision to Unknown Object (????).Sep 12 2013, 5:51 AM

Changes per review:

  • FeatureSHA implies FeatureSSE2
  • Use i128mem (oops)
  • Add sources and constraints for dest registers.

I forgot, the mem form needs mayLoad=1. Otherwise LGTM.

Thanks! Committed as r190611.

From: Craig Topper [mailto:craig.topper@gmail.com]
Sent: Thursday, September 12, 2013 11:24 AM
To: reviews+D1650+public+96a0704a61516414@llvm-reviews.chandlerc.com
Cc: nrotem@apple.com; Langmuir, Ben; llvm-commits@cs.uiuc.edu
Subject: Re: [PATCH] Partial support for Intel SHA Extensions (sha1rnds4)

I forgot, the mem form needs mayLoad=1. Otherwise LGTM.

benlangmuir abandoned this revision.Sep 19 2013, 7:47 AM