Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update lpc_intrin_sse.c #33

Closed
wants to merge 1 commit into from
Closed

Update lpc_intrin_sse.c #33

wants to merge 1 commit into from

Conversation

Shark64
Copy link
Contributor

@Shark64 Shark64 commented Jun 22, 2017

Minor change, remove redundant assignment before broadcasting into xmm registers

Minor change, remove redundant assignment before broadcasting into xmm registers
@erikd
Copy link
Member

erikd commented Jun 23, 2017

@lvqcl What do you think? The Travis failure was a timeout and hence should be ignored (but I have restarted it).

@lvqcl
Copy link
Contributor

lvqcl commented Jun 26, 2017

I tested MSVS 2105 and GCC 7.1, and they generate exactly the same code before and after this change, so it should be OK.

@erikd
Copy link
Member

erikd commented Jun 26, 2017

Thanks @lvqcl for reviewing.

Thanks @Shark64 for the patch. I've manually merged it into the Xiph.org git tree which will be mirrored here in the next 24 or so hours.

@erikd erikd closed this Jun 26, 2017
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.

None yet

3 participants