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

2 x slower with LLVM 4.X - 5.X #18

Open
9il opened this Issue Mar 30, 2017 · 12 comments

Comments

Projects
None yet
4 participants
@9il
Member

9il commented Mar 30, 2017

LLVM 4.0, avx512f has the same issue as LLVM 4.0, broadwell.

LDC - the LLVM D compiler (1.1.1):
  based on DMD v2.071.2 and LLVM 4.0.0
  built with LDC - the LLVM D compiler (0.17.3)
  Default target: x86_64-apple-darwin16.4.0
  Host CPU: haswell
  http://dlang.org - http://wiki.dlang.org/LDC

vs

LDC - the LLVM D compiler (1.1.1):
  based on DMD v2.071.2 and LLVM 3.9.1
  built with LDC - the LLVM D compiler (1.1.1)
  Default target: x86_64-apple-darwin16.4.0
  Host CPU: haswell
  http://dlang.org - http://wiki.dlang.org/LDC

@9il 9il added the bug label Mar 30, 2017

@JohanEngelen

This comment has been minimized.

JohanEngelen commented Mar 30, 2017

Could you spend some time on minimizing this? Thanks!

@9il

This comment has been minimized.

Member

9il commented Mar 31, 2017

Could you spend some time on minimizing this? Thanks!

Done, please see gists list in the issue.

@klickverbot

This comment has been minimized.

klickverbot commented Mar 31, 2017

@9il: What would probably be most useful is to generate the LLVM IR for the kernel function (-output-ll) and post it at bugs.llvm.org together with the assembler listings, possibly CCing some of the people who have worked on x86 codegen before (if you happen to know someone). Am I correct in the assumption that simple is just a manually expanded version of generic?

@9il

This comment has been minimized.

Member

9il commented Mar 31, 2017

Am I correct in the assumption that simple is just a manually expanded version of generic?

Yes.

@9il

This comment has been minimized.

Member

9il commented Mar 31, 2017

I have requested account at bugs.llvm.org

@klickverbot

This comment has been minimized.

klickverbot commented Mar 31, 2017

Okay, thanks. If there are any issues with the account, I can also make the report for you.

@klickverbot

This comment has been minimized.

klickverbot commented Apr 6, 2017

Any update on this? And does LLVM 4.0 only decrease performance on your code or are there improvements as well? We'll have to decide on the compiler to use to ship the pre-built 1.2 binaries soon.

@9il

This comment has been minimized.

Member

9il commented Apr 8, 2017

Only LLVM 4.0 has no bad AVX512 support. Feel free to update compiler to the LLVM 4.0. GLAS is not completely implemented for BLAS API. I will fill the issue and send link to the LDC gitter

@WebDrake

This comment has been minimized.

WebDrake commented Sep 11, 2017

Any news on whether v4.0.1 or v5.0.0 fix this?

@9il

This comment has been minimized.

Member

9il commented Sep 12, 2017

I can not compile LDC with LLVM 5.0 on macos

@klickverbot

This comment has been minimized.

klickverbot commented Sep 12, 2017

@9il: See Gitter – you might need to turn off detection of LLD headers explicitly with LLVM 5.0 until this is fixed.

@9il 9il changed the title from 2 x slower with LLVM 4.0 to 2 x slower with LLVM 4.X - 5.X Oct 27, 2017

@9il

This comment has been minimized.

Member

9il commented Oct 27, 2017

Any news on whether v4.0.1 or v5.0.0 fix this?

It is not fixed in v5.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment