Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Exclude AVX- and SSE-using source files from non-x86 builds #564

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Exclude AVX- and SSE-using source files from non-x86 builds #564

wants to merge 2 commits into from

Conversation

lexi-nadia
Copy link

No description provided.

@lexi-nadia
Copy link
Author

This is a second attempt. This PR does not use target_compatible_with; instead, it only uses select, which is supported by the version of Bazel that qsim currently uses.

@95-martin-orion 95-martin-orion added the kokoro:run Trigger Kokoro builds for this PR. label Sep 26, 2022
@qsim-qsimh-bot qsim-qsimh-bot removed the kokoro:run Trigger Kokoro builds for this PR. label Sep 26, 2022
@95-martin-orion
Copy link
Collaborator 

ERROR: Analysis of target '//apps:qsim_amplitudes' failed; build aborted:
       no such package '@platforms//cpu': The repository '@platforms' could not be resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@95-martin-orion 95-martin-orion Awaiting requested review from 95-martin-orion

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lexi-nadia @95-martin-orion @qsim-qsimh-bot