Bug 212483
| Summary: | devel/llvm38: Add compiler-rt support on arm64 | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Andrew Turner <Andrew> | ||||||
| Component: | Individual Port(s) | Assignee: | Brooks Davis <brooks> | ||||||
| Status: | Closed Overcome By Events | ||||||||
| Severity: | Affects Some People | CC: | emaste, linimon | ||||||
| Priority: | --- | Keywords: | feature, patch | ||||||
| Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(brooks) |
||||||
| Hardware: | arm64 | ||||||||
| OS: | Any | ||||||||
| Bug Depends on: | |||||||||
| Bug Blocks: | 201763 | ||||||||
| Attachments: |
|
||||||||
Created attachment 174519 [details]
Log from llvm 3.8.1
@Andrew, given the WIP'ness of the patch, perhaps a review is more appropriate until ready? Re-add blocked issue (mid-air collision) I created a bug as it's easier to share log files that will be needed to complete the patch. -DCOMPILER_RT_BUILD_SANITIZERS=off seems wrong. If the sanitizers don't build, there's no point in trying to enable COMPILER_RT since that's all the knob does. What state is this in, with recent devel/llvm* versions? This should likely be closed. I don't know what is required to enable sanitizers on newer versions. It may be that some just work. It got stuck in review https://reviews.llvm.org/D32175 I'm closing this for now. We can work on this after upstream accepts the changes. |
Created attachment 174518 [details] WIP patch The llvm ports don't have compiler-rt support. The attached patch starts to add this, but needs some work before it could be committed.