Skip to content

Build fixes (both bazel and cmake)#28

Merged
vimarsh6739 merged 46 commits intomainfrom
vim/bazelfix
Feb 23, 2026
Merged

Build fixes (both bazel and cmake)#28
vimarsh6739 merged 46 commits intomainfrom
vim/bazelfix

Conversation

@vimarsh6739
Copy link
Member

@vimarsh6739 vimarsh6739 commented Feb 17, 2026

@vimarsh6739 vimarsh6739 marked this pull request as ready for review February 17, 2026 09:27
@vimarsh6739 vimarsh6739 requested a review from wsmoses February 17, 2026 09:27
@vimarsh6739 vimarsh6739 changed the title Fix static build for enzyme-clang More bazel fixups Feb 17, 2026
#include "llvm/Analysis/PostDominators.h"

#include "PreserveNVVM.h"
#include "Enzyme/PreserveNVVM.h"
Copy link
Member

Choose a reason for hiding this comment

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

in a follow up pr we should rename these files/directories

Copy link
Member

@wsmoses wsmoses left a comment

Choose a reason for hiding this comment

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

lgtm but let's wait for ci to pass before merge

@vimarsh6739
Copy link
Member Author

hmm looks like a linker issue of some kind, let me try to repro

@vimarsh6739 vimarsh6739 merged commit 4529ef0 into main Feb 23, 2026
2 checks passed
@vimarsh6739 vimarsh6739 deleted the vim/bazelfix branch February 23, 2026 00:37
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.

2 participants