lowRISC / lowRISC/riscv-llvm

Support TLS

Open
#44 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

completeness llvm
Dominant language
Shell
Stars
277
Forks
55
PR merge metrics
No merged PRs in 30d

Description

I've migrated a D codebase from the riscv/riscv-llvm -based compiler to the lowRISC/riscv-llvm one. After the latest lowRISC/riscv-llvm changes I was able to remove essentially all workarounds for invalid code (although the latest changes (because of the new branch analysis?) seemed to cause a 4X performance penalty, despite allowing more modules to compile without -O0).

The only major issue remaining for great D support in this LLVM backend is the lack of thread-local storage. Since D globals default to TLS, that lack is particularly felt. I'm opening this issue to communicate the relative importance of adding TLS. Please address that when possible.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the riscv-llvm backend and investigate how thread-local storage is represented and supported in the LLVM-based compiler. Use the issue's D globals and comparison with riscv/riscv-llvm as context, then determine the required implementation scope and validation needed for TLS support.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.