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

Use ExtVec on PPC64LE #12390

Merged
merged 1 commit into from Nov 12, 2015
Merged

Conversation

davidlt
Copy link
Contributor

@davidlt davidlt commented Nov 12, 2015

Treat PPC64LE in a same way as ARMv7/ARMv8/MIC architectures.
Tested on fc22_ppc64le_gcc493.

Signed-off-by: David Abdurachmanov davidlt@cern.ch

Treat PPC64LE in a same way as ARMv7/ARMv8/MIC architectures.
Tested on fc22_ppc64le_gcc493.

Signed-off-by: David Abdurachmanov <davidlt@cern.ch>
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @davidlt for CMSSW_8_0_X.

It involves the following packages:

DataFormats/Math

@cmsbuild, @cvuosalo, @davidlange6, @slava77 can you please review it and eventually sign? Thanks.
@makortel this is something you requested to watch as well.

Following commands in first line of a comment are recognized

  • +1|approve[d]|sign[ed]: L1/L2's to approve it
  • -1|reject[ed]: L1/L2's to reject it
  • assign <category>[,<category>[,...]]: L1/L2's to request signatures from other categories
  • unassign <category>[,<category>[,...]]: L1/L2's to remove signatures from other categories
  • hold: L1/L2's to mark it as on hold
  • merge: L1 to merge this request
  • [@cmsbuild,] please test: L1/L2 and selected users to start jenkins tests
  • [@cmsbuild,] please test with cms-sw/cmsdist#<PR>: L1/L2 and selected users to start jenkins tests using externals from cmsdist

@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@slava77
Copy link
Contributor

slava77 commented Nov 12, 2015

@cmsbuild please test

@slava77
Copy link
Contributor

slava77 commented Nov 12, 2015

@Degano @smuzaffar @davidlt
Is there a jenkins way of testing on a different arch? (cross-compiling or going to a machine with a corresponding arch/OS)

@cmsbuild
Copy link
Contributor

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9681/console

@davidlt
Copy link
Contributor Author

davidlt commented Nov 12, 2015

Not yet. Once ROOT 6.04 or/and 6.06 becomes a default on CMSSW, then we will look into that. For POWER8 I am working on bootstrapping it, thus it has no RPMs yet in our repositories. I don't think there is a high number of POWER8 machines at CERN at this point also.

@cmsbuild
Copy link
Contributor

-1

Tested at: f7fd913
I found errors in the following unit tests:

---> test testDataFormatsPatCandidates had ERRORS
---> test runtestSimCalorimetryHGCalSimProducers had ERRORS

you can see the results of the tests here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-12390/9681/summary.html

@davidlt
Copy link
Contributor Author

davidlt commented Nov 12, 2015

Failures not related as the change does not affect x86_64 in any way.

@cmsbuild
Copy link
Contributor

@slava77
Copy link
Contributor

slava77 commented Nov 12, 2015

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_8_0_X IBs (but tests are reportedly failing). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @smuzaffar

davidlt added a commit that referenced this pull request Nov 12, 2015
@davidlt davidlt merged commit 451075d into cms-sw:CMSSW_8_0_X Nov 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants