google / google/heir

Clean up OpenMP gates

Open
#3,103 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
MLIR
Stars
906
Forks
171
Avg merge
4d 12h
Merged PRs (30d)
32

Description

I believe after the hermetic LLVM toolchain is merged, we can remove all of the gates around `openmp`, since the hermetic toolchain has native OpenMP support.

- Remove enable_openmp bazel flags
- Simplify CI to not need to install openmp (except for workflows that build and test OpenFHE for the frontend)

Contributor guide

Open the contributing guide

Research direction

Search the Bazel configuration and CI workflows for `enable_openmp`, `openmp`, and OpenFHE frontend jobs. Confirm the hermetic LLVM toolchain prerequisite first, then remove the Bazel gates and simplify OpenMP installation while retaining it for workflows that build and test OpenFHE for the frontend.

Written by the indexing model from the issue text.

Assessment

Domain
build-system, ci-cd
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.