[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

Segher Boessenkool  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #8 from Segher Boessenkool  ---
Fixed on all open release branches.

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

--- Comment #7 from Segher Boessenkool  ---
Author: segher
Date: Thu Oct 17 19:52:55 2019
New Revision: 277132

URL: https://gcc.gnu.org/viewcvs?rev=277132=gcc=rev
Log:
Backport from trunk
2019-03-15  Segher Boessenkool  

PR rtl-optimization/89721
* lra-constraints (invariant_p): Return false if side_effects_p holds.

Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/lra-constraints.c

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

--- Comment #6 from Segher Boessenkool  ---
Author: segher
Date: Thu Oct 17 19:51:01 2019
New Revision: 277131

URL: https://gcc.gnu.org/viewcvs?rev=277131=gcc=rev
Log:
Backport from trunk
2019-03-15  Segher Boessenkool  

PR rtl-optimization/89721
* lra-constraints (invariant_p): Return false if side_effects_p holds.

Modified:
branches/gcc-8-branch/gcc/ChangeLog
branches/gcc-8-branch/gcc/lra-constraints.c

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

--- Comment #5 from Segher Boessenkool  ---
No, it needs backports.  Thanks for reminding me!

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[bergner at gcc dot gnu.org]

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

--- Comment #4 from Peter Bergner  ---
Segher, can we move this to Fixed now?

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

Segher Boessenkool  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |segher at gcc dot 
gnu.org

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

--- Comment #3 from Segher Boessenkool  ---
Fixed on trunk so far.

[Bug rtl-optimization/89721] __builtin_mffs sometimes optimized away

[segher at gcc dot gnu.org]

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

--- Comment #2 from Segher Boessenkool  ---
Author: segher
Date: Fri Mar 15 22:09:15 2019
New Revision: 269716

URL: https://gcc.gnu.org/viewcvs?rev=269716=gcc=rev
Log:
LRA: side_effects_p stmts' output is not invariant (PR89721)

PR89721 shows LRA treating an unspec_volatile's result as invariant,
which of course isn't correct.  This patch fixes it.


PR rtl-optimization/89721
* lra-constraints (invariant_p): Return false if side_effects_p holds.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/lra-constraints.c