Skip to content

Fix LLVM default CPU on powerpc64 and powerpc64le#30987

Merged
bors merged 1 commit into
rust-lang:masterfrom
antonblanchard:powerpc64_merge3
Jan 18, 2016
Merged

Fix LLVM default CPU on powerpc64 and powerpc64le#30987
bors merged 1 commit into
rust-lang:masterfrom
antonblanchard:powerpc64_merge3

Conversation

@antonblanchard
Copy link
Copy Markdown
Contributor

We currently pass generic as the CPU to LLVM. This results in worse
than required code generation. On little endian, which is only POWER8,
we avoid many POWER4 and newer instructions.

Pass ppc64 and ppc64le instead.

We currently pass generic as the CPU to LLVM. This results in worse
than required code generation. On little endian, which is only POWER8,
we avoid many POWER4 and newer instructions.

Pass ppc64 and ppc64le instead.
@rust-highfive
Copy link
Copy Markdown
Contributor

r? @nrc

(rust_highfive has picked a reviewer for you, use r? to override)

@nrc
Copy link
Copy Markdown
Member

nrc commented Jan 17, 2016

r? @alexcrichton

@rust-highfive rust-highfive assigned alexcrichton and unassigned nrc Jan 17, 2016
@alexcrichton
Copy link
Copy Markdown
Member

@bors: r+ 1fb9f42

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jan 18, 2016

⌛ Testing commit 1fb9f42 with merge acc699f...

bors added a commit that referenced this pull request Jan 18, 2016
We currently pass generic as the CPU to LLVM. This results in worse
than required code generation. On little endian, which is only POWER8,
we avoid many POWER4 and newer instructions.

Pass ppc64 and ppc64le instead.
bors added a commit that referenced this pull request Jan 18, 2016
@bors bors merged commit 1fb9f42 into rust-lang:master Jan 18, 2016
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.

5 participants