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

Add support for LLVM 12 #3663

Merged
merged 2 commits into from
Feb 13, 2021
Merged

Add support for LLVM 12 #3663

merged 2 commits into from
Feb 13, 2021

Conversation

kinke
Copy link
Member

@kinke kinke commented Feb 11, 2021

No description provided.

@kinke kinke force-pushed the llvm12 branch 4 times, most recently from abfb69e to d9c8b38 Compare February 12, 2021 06:14
@kinke kinke changed the title Add initial support for LLVM 12 Add support for LLVM 12 Feb 12, 2021
@kinke kinke marked this pull request as ready for review February 12, 2021 06:14
AFAIK, the old port wasn't usable with LDC anyway, so eagerly switch
to the new one; the guys think it's mature enough to be the default
for `ld64.lld`:

https://lists.llvm.org/pipermail/llvm-dev/2021-January/147665.html
@kinke
Copy link
Member Author

kinke commented Feb 12, 2021

Tested with current vanilla release/12.x branch (rc1+) on Linux x64; all tests pass except for druntime-test-profile which I haven't looked into. There's a severe issue wrt. emitting bitcode files though - e.g., ldc2 -output-bc .../druntime/src/core/cpuid.d hits an assertion deep inside LLVM (-output-{o,ll} work fine). Other druntime modules worked, so I suspect once again an issue related to inline asm.

@kinke
Copy link
Member Author

kinke commented Feb 13, 2021

Merging eagerly as anticipation for distro packagers requiring LLVM 12 support for v1.25 once the LLVM final is out.

@kinke kinke merged commit fac87ee into ldc-developers:master Feb 13, 2021
@kinke kinke deleted the llvm12 branch February 13, 2021 00:48
kinke added a commit to kinke/ldc that referenced this pull request Apr 11, 2021
I've missed this in ldc-developers#3663 (only tested manually on Linux back then).
kinke added a commit that referenced this pull request Apr 17, 2021
I've missed this in #3663 (only tested manually on Linux back then).
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.

1 participant