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.

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

x86::tbm::bextri triggers an LLVM codegen error #26

Closed
gnzlbg opened this issue Sep 19, 2017 · 3 comments
Closed

x86::tbm::bextri triggers an LLVM codegen error #26

gnzlbg opened this issue Sep 19, 2017 · 3 comments
Labels
A-x86

Comments

@gnzlbg
Copy link
Contributor

gnzlbg commented Sep 19, 2017

It is the only instruction not implemented from the TBM feature set.
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
3ce2f12 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
9621515 
Closes rust-lang#26 .
@gnzlbg
Copy link
Contributor Author

gnzlbg commented Nov 6, 2017 •
edited
Loading

@alexcrichton @BurntSushi the root of the issue is that tbm::bextri takes an argument (u32/u64) that must be immediate. Lucky for us, only the first 16 bits of this value matter. Unlucky for us, using a constify_imm16!, with a match statement containing 65534 arms, makes the stdsimd crate take between forever and never to compile 😆

Thoughts?

@alexcrichton
Copy link
Member

I'd probably just leave this open for now, I'm not sure how we'd fix it :(

gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
a303fba 
Closes rust-lang#26 .
@gnzlbg gnzlbg mentioned this issue Nov 6, 2017
Closed
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
14fd03d 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
724f459 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
d48a21e 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
8cad6f9 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 6, 2017
@gnzlbg
[tbm] add bextri
a9feed7 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 7, 2017
@gnzlbg
[tbm] add bextri
dfc943b 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 7, 2017
@gnzlbg
[tbm] add bextri
b40cbdf 
Closes rust-lang#26 .
gnzlbg added a commit to gnzlbg/stdsimd that referenced this issue Nov 7, 2017
@gnzlbg
[tbm] add bextri
7cb8c0c 
Closes rust-lang#26 .
@gnzlbg gnzlbg mentioned this issue Dec 22, 2017
Closed
@alexcrichton
Copy link
Member

I'm gonna close this in favor of #248 as I believe they're basically the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-x86
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexcrichton @gnzlbg