Skip to content

Remove dead code related to PPC Big Endian #4107

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Remove dead code related to PPC Big Endian #4107

wants to merge 1 commit into from

Conversation

jjhursey
Copy link
Member

@jjhursey
ppc BE: Start removing code
50294ac 
Signed-off-by: Joshua Hursey <jhursey@us.ibm.com>
@rhc54
Copy link
Contributor

rhc54 commented Aug 17, 2017

@jjhursey Do we need to port the configure changes relating to #4102 to the PMIx code as well?

@jjhursey
Copy link
Member Author

@rhc54 Yeah we should. I'll take point on that. It should be a slight variation on the patch in #4053

@hjelmn
Copy link
Member

hjelmn commented Mar 21, 2018

I think the consensus is to bring back BE support.

@hppritcha
Copy link
Member

@jjhursey what's happening with this PR? Should it be closed?

@jsquyres
Copy link
Member

Yes, per #4349 (comment), we should close this PR without merging.

@jjhursey jjhursey closed this Mar 22, 2018
@jjhursey jjhursey deleted the cleanup/rm-ppc-be branch March 22, 2018 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Target: main ⚠️ WIP-DNM!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jjhursey @rhc54 @hjelmn @hppritcha @jsquyres @bwbarrett