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

Merge with fixes of 308c45e6 (9) #233

Merged
merged 5 commits into from
Aug 15, 2024

Conversation

mgehre-amd
Copy link
Collaborator

No description provided.

Xinyu Yang and others added 3 commits April 11, 2024 19:48
Fix the case PrimListUnpackOp's result num is not equal to PrimList
length.
See the following example:
```python
    def forward(self, x):
        if len(x.shape) == 5:
            b0, t, c0, h0, w0 = x.shape
            b, c, h, w = torch.mul(b0, t), c0, h0, w0
        else:
            b1, c1, h1, w1 = x.shape
            b, c, h, w = b1, c1, h1, w1
        res = torch.reshape(x, [b, c, h, w])
        return res
```
Without this fix, the following error message will occur:
```
/root/torch-mlir/externals/llvm-project/mlir/lib/IR/PatternMatch.cpp:118: virtual void mlir::RewriterBase::replaceOp(mlir::Operation *, mlir::ValueRange): Assertion `op->getNumResults() == newValues.size() && "incorrect # of replacement values"' failed.
```
@mgehre-amd mgehre-amd requested a review from cmcgirr-amd August 14, 2024 09:31
Base automatically changed from bump_to_6524838b to feature/backport_ea1_ops August 15, 2024 09:34
@mgehre-amd mgehre-amd enabled auto-merge August 15, 2024 10:14
@mgehre-amd mgehre-amd requested review from cferry-AMD and removed request for cmcgirr-amd August 15, 2024 10:14
@mgehre-amd mgehre-amd merged commit 1affa1f into feature/backport_ea1_ops Aug 15, 2024
3 checks passed
@mgehre-amd mgehre-amd deleted the bump_to_308c45e6 branch August 15, 2024 12:10
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.

3 participants