Skip to content

update llvm and stablehlo #3232

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 5 commits into from
Jul 24, 2025
Merged

update llvm and stablehlo #3232

merged 5 commits into from
Jul 24, 2025

Conversation

jagadeeshvx
Copy link
Contributor

@jagadeeshvx jagadeeshvx commented Jul 18, 2025

This PR updates llvm to commit: f8cb7987c64dcffb72414a40560055cb717dbf74
Stablehlo commit: 69d6dae46e1c7de36e6e6973654754f05353cba5

Changes:

  1. Fixed the build warning: llvm/include/llvm/ADT/ArrayRef.h:70:18: note: 'ArrayRef' has been explicitly marked deprecated here
    70 | /implicit/ LLVM_DEPRECATED("Use {} or ArrayRef() instead", "{}")
  2. Fixed the Build failure: Switch type and value ordering for arith Constant[XX]Op

Signed-off-by: jagadeeshvx <jagadeesh.vijayendran@multicorewareinc.com>
@jenkins-droid
Copy link
Collaborator

Can one of the admins verify this patch?

@jagadeeshvx
Copy link
Contributor Author

currently working on onnx builds warning.

@Sunny-Anand
Copy link
Collaborator

@jenkins-droid test this please.

Copy link
Collaborator

@Sunny-Anand Sunny-Anand left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: jagadeeshvx <jagadeesh.vijayendran@multicorewareinc.com>
@jenkins-droid
Copy link
Collaborator

Can one of the admins verify this patch?

@jagadeeshvx jagadeeshvx requested a review from Sunny-Anand July 19, 2025 05:59
@jenkins-droid
Copy link
Collaborator

Can one of the admins verify this patch?

@Sunny-Anand
Copy link
Collaborator

@jenkins-droid test this please.

@jenkins-droid
Copy link
Collaborator

Can one of the admins verify this patch?

@tungld
Copy link
Collaborator

tungld commented Jul 24, 2025

@jenkins-droid test this please

1 similar comment
@tungld
Copy link
Collaborator

tungld commented Jul 24, 2025

@jenkins-droid test this please

@Sunny-Anand Sunny-Anand merged commit 096ae7b into onnx:main Jul 24, 2025
7 checks passed
@jenkins-droid
Copy link
Collaborator

Jenkins Linux s390x Build #17104 [push] update llvm and stablehl... started at 15:53

@jenkins-droid
Copy link
Collaborator

Jenkins Linux ppc64le Build #16048 [push] update llvm and stablehl... started at 15:53

@jenkins-droid
Copy link
Collaborator

Jenkins Linux amd64 Build #17102 [push] update llvm and stablehl... started at 14:54

@jenkins-droid
Copy link
Collaborator

Jenkins Linux s390x Build #17104 [push] update llvm and stablehl... passed after 2 hr 43 min

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.

4 participants