Skip to content
This repository was archived by the owner on Mar 25, 2025. It is now read-only.

[LLVM][refactoring] Preparing code for builders#983

Merged
iomaganaris merged 8 commits intollvmfrom
george/builders
Jan 2, 2023
Merged

[LLVM][refactoring] Preparing code for builders#983
iomaganaris merged 8 commits intollvmfrom
george/builders

Conversation

@georgemitenkov
Copy link
Copy Markdown
Collaborator

Since #911 was a huge change and difficult to review, it is better to split into multiple PRs. This one reorganises code, fixes some typos ad renames functions. Some code has now explicit if (GPU) else if (SIMD) else blocks which is what we need to have multiple builders.

@bbpbuildbot
Copy link
Copy Markdown
Collaborator

Logfiles from GitLab pipeline #92357 (:no_entry:) have been uploaded here!

Status and direct links:

@iomaganaris
Copy link
Copy Markdown
Contributor

FYI the Azure CI are failing due to incompatibilities between the llvm branch of NMODL and the latest version of NEURON being used. I would only focus on the GitLab CI that runs the CPU and GPU integration tests of the JIT workflow

@bbpbuildbot
Copy link
Copy Markdown
Collaborator

Logfiles from GitLab pipeline #92578 (:white_check_mark:) have been uploaded here!

Status and direct links:

@iomaganaris iomaganaris merged commit 4b4138a into llvm Jan 2, 2023
@iomaganaris iomaganaris deleted the george/builders branch January 2, 2023 21:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants