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

Bump to 647d75d3a883 (99) #254

Closed
wants to merge 5 commits into from
Closed

Conversation

mgehre-amd
Copy link
Collaborator

@mgehre-amd mgehre-amd commented Aug 13, 2024

No description provided.

@mgehre-amd mgehre-amd requested a review from cferry-AMD August 13, 2024 11:27
Base automatically changed from bump_to_19266ca3 to feature/fused-ops August 15, 2024 07:01
An error occurred while trying to automatically change base from bump_to_19266ca3 to feature/fused-ops August 15, 2024 07:01
@mgehre-amd mgehre-amd marked this pull request as draft August 15, 2024 07:03
@mgehre-amd mgehre-amd changed the title Bump to 647d75d3a883 (7) Bump to 647d75d3a883 (3) Aug 15, 2024
@mgehre-amd mgehre-amd changed the base branch from feature/fused-ops to bump_to_516ccce7fa26 August 15, 2024 07:13
Base automatically changed from bump_to_516ccce7fa26 to feature/fused-ops August 15, 2024 12:00
@mgehre-amd mgehre-amd changed the base branch from feature/fused-ops to matthias.bump_to_d84252e064b3 August 15, 2024 12:49
@mgehre-amd mgehre-amd changed the title Bump to 647d75d3a883 (3) Bump to 647d75d3a883 (99) Aug 15, 2024
Base automatically changed from matthias.bump_to_d84252e064b3 to feature/fused-ops August 15, 2024 15:48
An error occurred while trying to automatically change base from matthias.bump_to_d84252e064b3 to feature/fused-ops August 15, 2024 15:48
@mgehre-amd mgehre-amd changed the base branch from feature/fused-ops to bump_to_647d75d3 August 15, 2024 16:17
@mgehre-amd mgehre-amd closed this Aug 15, 2024
mgehre-amd pushed a commit that referenced this pull request Aug 21, 2024
…ndling (llvm#76644)

Fold BICi if all destination bits are already known to be zeroes

```llvm
define <8 x i16> @haddu_known(<8 x i8> %a0, <8 x i8> %a1) {
  %x0 = zext <8 x i8> %a0 to <8 x i16>
  %x1 = zext <8 x i8> %a1 to <8 x i16>
  %hadd = call <8 x i16> @llvm.aarch64.neon.uhadd.v8i16(<8 x i16> %x0, <8 x i16> %x1)
  %res = and <8 x i16> %hadd, <i16 511, i16 511, i16 511, i16 511,i16 511, i16 511, i16 511, i16 511>
  ret <8 x i16> %res
}
declare <8 x i16> @llvm.aarch64.neon.uhadd.v8i16(<8 x i16>, <8 x i16>)
```

```
haddu_known:                            // @haddu_known
        ushll   v0.8h, v0.8b, #0
        ushll   v1.8h, v1.8b, #0
        uhadd   v0.8h, v0.8h, v1.8h
        bic     v0.8h, #254, lsl #8 <-- this one will be removed as we know high bits are zero extended
        ret
```

Fixes llvm#53881
Fixes llvm#53622
@mgehre-amd mgehre-amd deleted the bump_to_647d75d3a883 branch September 11, 2024 13:45
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.

2 participants