-
Notifications
You must be signed in to change notification settings - Fork 373
start of llvm/stablehlo bump #3302
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
start of llvm/stablehlo bump #3302
Conversation
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits. LGTM
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
|
Hi @gongsu832, could you please help me debug the s390x absl and protobuf errors? These only started occurring after the merge. Thanks! fyi.. @AlexandreEichenberger |
|
Here is the errors from the log on my Mac, I just had to Thanks @christopherlmunoz for doing the update. |
|
Can Abseil be installed on the s390x test instance? I believe since this is the first LLVM bump |
|
@jenkins-droid test this please |
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
|
@jenkins-droid test this please |
|
Hi @AlexandreEichenberger and @gongsu832, Separate onnx/protobuf issue. All PRs are now failing on s390x because |
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
…3307) Signed-off-by: Alexandre Eichenberger <alexe@us.ibm.com> Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
|
@jenkins-droid test this please |
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Christopher Munoz <chrismunoz1019@gmail.com>
Signed-off-by: Tung D. Le <tung@jp.ibm.com>
|
@jenkins-droid test this please |
1 similar comment
|
@jenkins-droid test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM,
There seems to be lots of extract / insert vector operations around the onnx.round operations; this will need a separate investigation to see if they can be avoided.
|
Jenkins Linux s390x Build #17566 [push] start of llvm/stablehlo ... started at 10:37 |
|
Jenkins Linux amd64 Build #17564 [push] start of llvm/stablehlo ... started at 09:37 |
|
Jenkins Linux s390x Build #17566 [push] start of llvm/stablehlo ... failed after 6 min 16 sec |
|
Jenkins Linux amd64 Build #17564 [push] start of llvm/stablehlo ... failed after 7 min 0 sec |
LLVM
113f01aa82d055410f22a9d03b3468fa68600589StableHLO
0a4440a5c8de45c4f9649bf3eb4913bf3f97da0dand fixes protobuf segfault build errors, see onnx issue: (onnx/onnx#7402).
pip install onnx==1.17.0 --no-build-isolation