Skip to content

Conversation

@jorickert
Copy link

@jorickert jorickert commented Feb 3, 2025

No description provided.

nikic and others added 2 commits November 1, 2024 16:23
This enables the assertion introduced in
llvm#106524, which checks that the
value passed to the constructor is indeed a valid N-bit signed or
unsigned integer.

Places that previously violated the assertion were updated in advance,
e.g. in llvm#80309.

It is possible to opt-out of the check and restore the previous behavior
by setting implicitTrunc=true.

-----

The buildbot failures from the previous attempt should be fixed by
a18dd29 and
e2074c6.
@jorickert jorickert changed the title [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (8) [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (Needs torch bump) (8) Feb 3, 2025
@jorickert jorickert changed the title [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (Needs torch bump) (8) [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (Needs torch-mlir bump) (8) Feb 3, 2025
Base automatically changed from bump_to_b74e588e to bump_to_95c2d798 February 12, 2025 11:11
@jorickert jorickert changed the title [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (Needs torch-mlir bump) (8) [AutoBump] Merge with fixes of 3494ee95 (Nov 01) (Needs downstream fixes) (8) Feb 12, 2025
@jorickert
Copy link
Author

I can not observe failures in torch-mlir anymore, but multiple downstream tests assert when trying to construct invalid APInts

@mgehre-amd
Copy link
Collaborator

I updated torch-mlir to a newer version which explicitly sets implicitTrunc. Same approach can be used to fix downstream users.

Base automatically changed from bump_to_95c2d798 to feature/fused-ops February 13, 2025 08:26
@jorickert jorickert merged commit b31574a into feature/fused-ops Feb 18, 2025
3 of 4 checks passed
@jorickert jorickert deleted the bump_to_3494ee95 branch February 18, 2025 16:32
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.

6 participants