subject:"\[Bug gas\/20641\] powerpc\: Should not allow three\-operand cmp\[l\]\[i\]"

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-10-06 Thread amodra at gmail dot com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 Alan Modra changed: What|Removed |Added CC||benh at kernel dot crashing.org -- You

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-10-06 Thread segher at gcc dot gnu.org

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #9 from Segher Boessenkool --- Hi Markus, Yes, various things in the kernel were broken. You found another one. Note you are looking at an old kernel. Newer kernels have this (namely, the IDLE_STATE_ENTER_SEQ macro) in arch/pow

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-10-06 Thread markus at trippelsdorf dot de

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 Markus Trippelsdorf changed: What|Removed |Added CC||markus at trippelsdorf dot de -

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-10-05 Thread amodra at gmail dot com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 Alan Modra changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-09-29 Thread bergner at vnet dot ibm.com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #6 from Peter Bergner --- FYI, I just committed a fix to the gdb testsuite to account for these changes. * gdb.arch/powerpc-power.exp : Update tests to account for the compulsory L operand changes. * gdb.ar

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-09-28 Thread cvs-commit at gcc dot gnu.org

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #5 from cvs-commit at gcc dot gnu.org --- The master branch has been updated by Alan Modra : https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=a5721ba270ddf860e0e5a45bba456214e8eac2be commit a5721ba270ddf860e0e5a45bba456

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-09-27 Thread amodra at gmail dot com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #4 from Alan Modra --- dcbf needs to keep its L field optional. dcbf is used in the kernel and in glibc without L. Also tlbie and tlbiel ought to be optional, I think, because earlier versions of these insns lacked the L field.

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-09-27 Thread bergner at vnet dot ibm.com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #3 from Peter Bergner --- (In reply to Alan Modra from comment #2) > {"tbegin.", {HTM_R}}, > {"tsr.",{L}}, > > I'm inclined to say L should be optional when generating ppc32 code, for > cmp* and tlbie. Not optional everywhe

[Bug gas/20641] powerpc: Should not allow three-operand cmp[l][i]

2016-09-27 Thread amodra at gmail dot com

https://sourceware.org/bugzilla/show_bug.cgi?id=20641 --- Comment #2 from Alan Modra --- And there are quite a few other issues around L. L0 and L1 are unused so ought to disappear. dcbf, wclr L field is 2 bits, not 1. mtsle isn't documented in either Power ISA Version 3.0 or POWER8 UM v1.3.