Skip to content

Fix Sharpy build steps in README.md #7

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

Merged
merged 4 commits into from
Jun 25, 2024

Conversation

AllanZyne
Copy link
Collaborator

@AllanZyne AllanZyne commented Apr 30, 2024

  • Remove git checkout jit
  • Added a tip for building imex (need to add -DLLVM_ENABLE_RTTI=ON into cmake arguments)

fix #6

@AllanZyne AllanZyne changed the title Add IMEX Build Steps Fix IMEX Build Steps Apr 30, 2024
@AllanZyne AllanZyne changed the title Fix IMEX Build Steps Fix IMEX Build Steps in README.md Jun 24, 2024
README.md Outdated
@@ -9,7 +9,7 @@ Parallel and distributed execution currently is MPI/CSP-like. In a later version

## Setting up build environment

Install MLIR/LLVM and Intel® Extension for MLIR (IMEX, see https://github.com/intel/mlir-extensions). Besides, Add "-DLLVM_ENABLE_RTTI=ON" into cmake arguments, and build target is "all" (not "check-imex").
Install MLIR/LLVM and Intel® Extension for MLIR (IMEX, see https://github.com/intel/mlir-extensions). Besides, Add `-DLLVM_ENABLE_RTTI=ON` into cmake arguments, and build target is `all` (not `check-imex`).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

..... Make sure you use -DLLVM_ENABLE_RTTI=ON when configuring LLVM and use build build target all.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Done.

@AllanZyne AllanZyne changed the title Fix IMEX Build Steps in README.md Fix Sharpy build steps in README.md Jun 25, 2024
Copy link
Contributor

@fschlimb fschlimb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@fschlimb fschlimb merged commit f818327 into IntelPython:main Jun 25, 2024
2 checks passed
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.

IMEX Build Steps
2 participants