On 11/29/23 19:39, Joern Rennecke wrote:
On Wed, 29 Nov 2023 at 20:05, Joern Rennecke
<joern.renne...@embecosm.com> wrote:

I suspect it'd be more useful to add handling of LSHIFTRT and ASHIFTRT
.  Some ports do
a lot of static shifting.

+    case SS_ASHIFT:
+    case US_ASHIFT:
+      if (!mask || XEXP (x, 1) == const0_rtx)
+       return 0;

P.S.: I just realize that this is a pasto: in the case of a const0_rtx
shift count,
we returning 0 will usually be wrong.

I've attached my current patch version.
I would strongly suggest we not start adding a lot of new capabilities here, deferring such work until gcc-15. Our focus should be getting the existing code working well and cleanly implemented.

Jeff

Reply via email to