Skip to content

[bazel] Fix new CodeGen dep #91654

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 1 commit into from
May 9, 2024
Merged

Conversation

keith
Copy link
Member

@keith keith commented May 9, 2024

.../AMDGPUUtilsAndDesc/AMDGPUCallLowering.h:17:10: fatal error: 'llvm/CodeGen/GlobalISel/CallLowering.h' file not found

https://buildkite.com/llvm-project/upstream-bazel/builds/97166

````
.../AMDGPUUtilsAndDesc/AMDGPUCallLowering.h:17:10: fatal error: 'llvm/CodeGen/GlobalISel/CallLowering.h' file not found
```

https://buildkite.com/llvm-project/upstream-bazel/builds/97166
@keith keith requested review from AaronBallman and akuegel May 9, 2024 21:01
@keith keith requested a review from rupprecht as a code owner May 9, 2024 21:01
@llvmbot llvmbot added the bazel "Peripheral" support tier build system: utils/bazel label May 9, 2024
@keith
Copy link
Member Author

keith commented May 9, 2024

merging to unblock CI, please lmk if there are any issues and I'll fix in a follow up!

@keith keith merged commit 99f45b4 into llvm:main May 9, 2024
6 checks passed
@keith keith deleted the ks/bazel-fix-new-codegen-dep branch May 9, 2024 21:03
@akuegel
Copy link
Member

akuegel commented May 10, 2024

Thanks for the fix

bjacob pushed a commit to iree-org/llvm-project that referenced this pull request May 10, 2024
```
.../AMDGPUUtilsAndDesc/AMDGPUCallLowering.h:17:10: fatal error: 'llvm/CodeGen/GlobalISel/CallLowering.h' file not found
```

https://buildkite.com/llvm-project/upstream-bazel/builds/97166
ScottTodd added a commit to iree-org/iree that referenced this pull request May 10, 2024
…800a3 (#17330)

* torch-mlir integrated at bce800a.
* llvm-project integrated at 2083e97e plus local changes:
* Reverted llvm/llvm-project#89131 locally:
while this change is good in its own right, the `vector.interleave` that
it generates (instead of `vector.shuffle`) are not handled by some GPU
codegen lowerings.
        * Filed #17346.
* Cherry-picked Bazel build fix:
llvm/llvm-project#91654
* Several e2e tests have been temporarily disabled, follow-up work is
needed to reenable them: #17344

---------

Co-authored-by: MaheshRavishankar <[email protected]>
Co-authored-by: Scott Todd <[email protected]>
bjacob added a commit to iree-org/iree that referenced this pull request May 14, 2024
This allows dropping our existing local-revert of
llvm/llvm-project#89131 and cherry-pick of
llvm/llvm-project#91654 which we had introduced
in the earlier integrate #17330.

This locally reverts llvm/llvm-project#90802
because it causes numerical errors, reported at
llvm/llvm-project#90802 (comment).
bangtianliu pushed a commit to bangtianliu/iree that referenced this pull request Jun 5, 2024
…800a3 (iree-org#17330)

* torch-mlir integrated at bce800a.
* llvm-project integrated at 2083e97e plus local changes:
* Reverted llvm/llvm-project#89131 locally:
while this change is good in its own right, the `vector.interleave` that
it generates (instead of `vector.shuffle`) are not handled by some GPU
codegen lowerings.
        * Filed iree-org#17346.
* Cherry-picked Bazel build fix:
llvm/llvm-project#91654
* Several e2e tests have been temporarily disabled, follow-up work is
needed to reenable them: iree-org#17344

---------

Co-authored-by: MaheshRavishankar <[email protected]>
Co-authored-by: Scott Todd <[email protected]>
bangtianliu pushed a commit to bangtianliu/iree that referenced this pull request Jun 5, 2024
This allows dropping our existing local-revert of
llvm/llvm-project#89131 and cherry-pick of
llvm/llvm-project#91654 which we had introduced
in the earlier integrate iree-org#17330.

This locally reverts llvm/llvm-project#90802
because it causes numerical errors, reported at
llvm/llvm-project#90802 (comment).
LLITCHEV pushed a commit to LLITCHEV/iree that referenced this pull request Jul 30, 2024
…800a3 (iree-org#17330)

* torch-mlir integrated at bce800a.
* llvm-project integrated at 2083e97e plus local changes:
* Reverted llvm/llvm-project#89131 locally:
while this change is good in its own right, the `vector.interleave` that
it generates (instead of `vector.shuffle`) are not handled by some GPU
codegen lowerings.
        * Filed iree-org#17346.
* Cherry-picked Bazel build fix:
llvm/llvm-project#91654
* Several e2e tests have been temporarily disabled, follow-up work is
needed to reenable them: iree-org#17344

---------

Co-authored-by: MaheshRavishankar <[email protected]>
Co-authored-by: Scott Todd <[email protected]>
Signed-off-by: Lubo Litchev <[email protected]>
LLITCHEV pushed a commit to LLITCHEV/iree that referenced this pull request Jul 30, 2024
This allows dropping our existing local-revert of
llvm/llvm-project#89131 and cherry-pick of
llvm/llvm-project#91654 which we had introduced
in the earlier integrate iree-org#17330.

This locally reverts llvm/llvm-project#90802
because it causes numerical errors, reported at
llvm/llvm-project#90802 (comment).

Signed-off-by: Lubo Litchev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bazel "Peripheral" support tier build system: utils/bazel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants