Skip to content

Conversation

@huonw
Copy link
Contributor

@huonw huonw commented Aug 31, 2015

I believe everything that doesn't take a constant integer up to SSE4.2
should now be correct (I don't have any reason to believe that those
that do take constant integers are wrong; they're just more complicated
and I just haven't tested them in detail).

(Also, takes out two unused code paths from trans.)

@rust-highfive
Copy link
Contributor

r? @pnkfelix

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

@bors: r+ 423f256d02b3eaddd31bdd4c88d16199a62ac475

huonw added 2 commits August 31, 2015 18:33
I believe everything that doesn't take a constant integer up to SSE4.2
should now be correct (I don't have any reason to believe that those
that do take constant integers are wrong; they're just more complicated
and I just haven't tested them in detail).
@huonw
Copy link
Contributor Author

huonw commented Sep 1, 2015

@bors r=alexcrichton

@bors
Copy link
Collaborator

bors commented Sep 1, 2015

📌 Commit 787a21f has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Sep 1, 2015

⌛ Testing commit 787a21f with merge 4ad128b...

bors added a commit that referenced this pull request Sep 1, 2015
I believe everything that doesn't take a constant integer up to SSE4.2
should now be correct (I don't have any reason to believe that those
that do take constant integers are wrong; they're just more complicated
and I just haven't tested them in detail).

(Also, takes out two unused code paths from trans.)
@bors bors merged commit 787a21f into rust-lang:master Sep 1, 2015
@huonw huonw deleted the simd branch September 1, 2015 19:11
huonw added a commit to huonw/simd that referenced this pull request Sep 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants