[Bug tree-optimization/115026] msp430-elf fails gcc.dg/pr66444.c with prange enabled

[aldyh at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #4 from Aldy Hernandez  ---
Jeff, if you still have your tree around, could you try this patch?

I'll queue it with the rest of patches I will push before enabling prange when
the IPA issues are sorted out.

[Bug tree-optimization/115026] msp430-elf fails gcc.dg/pr66444.c with prange enabled

[aldyh at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #3 from Aldy Hernandez  ---
Created attachment 58169
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=58169&action=edit
proposed patch

[Bug tree-optimization/115026] msp430-elf fails gcc.dg/pr66444.c with prange enabled

[aldyh at gcc dot gnu.org via Gcc-bugs]

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

--- Comment #2 from Aldy Hernandez  ---
OK, this is embarrassing.

We are incorrectly folding a POINTER_PLUS_EXPR range operation:

 Folding statement: x_7 = 2048B + _2;
-Queued stmt for removal.  Folds to: 2062B
+Queued stmt for removal.  Folds to: 0B

The reason is that the prange::update_bitmask() code is ignoring its operand
and using the current bitmask.

I have no idea how this bootstrapped *any* architecture.  I suppose it needs a
value/mask pair (0xe / 0x0) for the second operand that actually indicates a
singleton.  *shrug*

Thanks for reporting this.

[Bug tree-optimization/115026] msp430-elf fails gcc.dg/pr66444.c with prange enabled

[aldyh at gcc dot gnu.org via Gcc-bugs]

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

Aldy Hernandez  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1
   Assignee|unassigned at gcc dot gnu.org  |aldyh at gcc dot gnu.org
   Last reconfirmed||2024-05-10

--- Comment #1 from Aldy Hernandez  ---
All mine baby!