Enable OpenMP for powerpc64
ClosedPublic
Actions

Authored by pkubaj on Sep 5 2019, 7:54 AM.

Details

Reviewers

alfredo
bdragon
dim
emaste

Group Reviewers

manpages

Commits

rS352763: Enable OpenMP for powerpc64

Summary

When powerpc64 switches to LLVM, use this patch to enable OpenMP as well. OpenMP on PPC is only for 64-bits, so don't make a 32-bit libomp. A change to openmp files is necesssary (under review on https://reviews.llvm.org/D67190), because it determines ELF format version based on endianness, which is incorrect.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

pkubaj created this revision.Sep 5 2019, 7:54 AM

Herald added a reviewer: manpages. · View Herald TranscriptSep 5 2019, 7:54 AM

Herald added subscribers: imp, bdrewery. · View Herald Transcript

OK from manpages.

Are you planning to patch llvm9 in ports?

Looks good to me, but I'm suspect :)

This revision is now accepted and ready to land.Sep 5 2019, 7:28 PM

In D21532#469129, @alfredo.junior_eldorado.org.br wrote:

Looks good to me, but I'm suspect :)

Merged to devel/llvm90 in https://svnweb.freebsd.org/ports?view=revision&revision=511878. We could also patch devel/llvm80, but llvm90 is going to be the default soon anyway.

Closed by commit rS352763: Enable OpenMP for powerpc64 (authored by dim). · Explain WhySep 26 2019, 6:24 PM

This revision was automatically updated to reflect the committed changes.

dim added a commit: rS352763: Enable OpenMP for powerpc64.