PPC64 patches

[yhcheng]

Changed file ksw.c with SIMD implementation for PPC64
Added a file called vec128int.h for SIMD functions.

[leecbaker]

Should this be committed upstream at https://github.com/attractivechaos/klib/ instead?

[yhcheng]

Thanks! I will do PR on the upstream.

[yhcheng]

submitted PR at upstream klib at attractivechaos/klib#72

[IvantheDugtrio]

I'm compiling this in CentOS 7 64-bit on a POWER7 machine and am getting an undeclared variable error in the big-endian section of vec128int.h
"vec128int.h.:169:35: error: '__vsr_left_half_long_long_in_memory' undeclared (first use in this function)
return (short) t.as_long_long[__vsr_left_half_long_long_in_memory];"

[yhcheng]

@IvantheDugtrio I added definition of __vsr_left_half_long_long_in_memory, it should fix it now

[IvantheDugtrio]

Thanks now it compiles perfectly!

[jlost]

Thanks, this also fixed my compilation on a POWER8 machine. Will this be pull request be accepted?

[IvantheDugtrio]

@yhcheng Do you have some documentation on vec128int.h? I'd like to expand the supported sse functions for use in porting other projects.

[wilwan01]

I also have a patch to enable BWA to compile for ARM, what’s the best way to contribute this patch upstream?

On 14/07/2016, 20:09, "James" <notifications@github.commailto:notifications@github.com> wrote:

Thanks, this also fixed my compilation on a POWER8 machine. Will this be pull request be accepted?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com//pull/71#issuecomment-232762707, or mute the threadhttps://github.com/notifications/unsubscribe/AITmGTWJ-oaddvQMXpu17xJ6hSFM1cFeks5qVolxgaJpZM4IzOyG.

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

[yhcheng]

@IvantheDugtrio Please check out https://www.ibm.com/developerworks/community/groups/community/powerveclib/

[jlost]

FYI, a bounty has been posted at https://www.bountysource.com/issues/35671464-fails-compile-on-power8-sse2-intrinsics-vsx-altivec for anyone who puts in the work to make sure this pull request is accepted, whether that means tracking down the owner and nudging them or making the code changes for it to pass review.

[YulongNiu]

@wilwan01 Could you please post details for ARM? Thanks!

[junaruga]

Perhaps, using simde for the SIMD implementation of other CPU architectures might be alternative way to build bwa.
It is used for bowtie2 to build bowtie2 on multiple CPU architectures.

[junaruga]

@wilwan01 Could you please post details for ARM? Thanks!

@YulongNiu Possibly here is the way to build bwa on ARM.
https://gitlab.com/arm-hpc/packages/-/wikis/packages/bwa

[junaruga]

I sent a PR to build bwa on ppc64le, arm64 and s390x here, #283 .
Maybe the build works on both ppc64le and ppc64 (big endian), as I used the Debian's way where some CPU architecture builds are enabled.