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.

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

Update asmjit submodule for OSS #1202

Closed
wants to merge 1 commit into from
Closed

Conversation

geyyer
Copy link
Contributor

@geyyer geyyer commented Jul 11, 2022

Differential Revision: D37763232

@netlify
Copy link

netlify bot commented Jul 11, 2022

Deploy Preview for eclectic-stroopwafel-199537 canceled.

Name Link
🔨 Latest commit 6964283
🔍 Latest deploy log https://app.netlify.com/sites/eclectic-stroopwafel-199537/deploys/62cd9f6e85500900088fe406

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37763232

@geyyer geyyer mentioned this pull request Jul 11, 2022
Summary:
Pull Request resolved: pytorch#1202

Update OSS asmjit version to d3fbf7c9bc7c1d1365a94a45614b91c5a3706b81, sync with the third-party/asmjit.

Add a -Wno-unused-variable compilation flag to enable Clang 14.

Reviewed By: brad-mengchi, jianyuh

Differential Revision: D37763232

fbshipit-source-id: e70e387408d0b851297465f66fc81d065a9a284c
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37763232

@geyyer geyyer force-pushed the export-D37763232 branch from 502df4d to 6964283 Compare July 12, 2022 16:21
@powderluv
Copy link

powderluv commented Jul 24, 2022

Can we please land this ? It fixes builds on latest versions of clang that is bundled in Ubuntu 22.04. See here for an example failure: https://github.com/llvm/torch-mlir/runs/7464486170?check_suite_focus=true#step:4:1663

@jianyuh
Copy link
Member

jianyuh commented Jul 28, 2022

@powderluv Landing the PR.

@powderluv
Copy link

Do you know who can update the Pytorch FBGEMM submodule. I think the automatic submodule updater stopped working and FBGEMM is pinned to something from three months ago. Happy to submit a PR if required.

@powderluv
Copy link

powderluv commented Jul 28, 2022

Here is a PR for updating PyTorch FBGEMM pytorch/pytorch#82396

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants