public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/113870] New: Add V2HF support
@ 2024-02-11  1:34 pinskia at gcc dot gnu.org
  2024-02-11  1:35 ` [Bug target/113870] " pinskia at gcc dot gnu.org
  2024-06-11 17:03 ` pinskia at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2024-02-11  1:34 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113870

            Bug ID: 113870
           Summary: Add V2HF support
           Product: gcc
           Version: 14.0
            Status: UNCONFIRMED
          Keywords: missed-optimization
          Severity: enhancement
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: pinskia at gcc dot gnu.org
  Target Milestone: ---
            Target: aarch64

Take:
```
void f( _Float16 * __restrict  a, float * __restrict b)
{

 b[0] = a[0];
 b[1] = a[1];
}

```
We should produce:
```
        ldr     s0, [x0]
        fcvtl   v0.4s, v0.4h
        str     d0, [x1]
        ret
```

But this requires V2HF support. I am not 100% sure if this is worth adding.
Adding V2HI and V4QI is worth it though (I have a patch for that).

There are not enough benchmarks that use _Float16 to say if it is worth it or
not.

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [Bug target/113870] Add V2HF support
  2024-02-11  1:34 [Bug target/113870] New: Add V2HF support pinskia at gcc dot gnu.org
@ 2024-02-11  1:35 ` pinskia at gcc dot gnu.org
  2024-06-11 17:03 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2024-02-11  1:35 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113870

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |113869

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Oh this depends on PR 113869 for the V4HF->V4SF pattern too :).


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113869
[Bug 113869] V4HF->V4SF pattern seems to be missing

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [Bug target/113870] Add V2HF support
  2024-02-11  1:34 [Bug target/113870] New: Add V2HF support pinskia at gcc dot gnu.org
  2024-02-11  1:35 ` [Bug target/113870] " pinskia at gcc dot gnu.org
@ 2024-06-11 17:03 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2024-06-11 17:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113870
Bug 113870 depends on bug 113869, which changed state.

Bug 113869 Summary: V4HF->V4SF pattern seems to be missing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113869

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-06-11 17:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-11  1:34 [Bug target/113870] New: Add V2HF support pinskia at gcc dot gnu.org
2024-02-11  1:35 ` [Bug target/113870] " pinskia at gcc dot gnu.org
2024-06-11 17:03 ` pinskia at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).