Hi Bill, On Fri, Jan 19, 2018 at 10:48:18PM -0600, Bill Schmidt wrote: > Here's another version of this patch incorporating the late-breaking news > that the AIX assembler doesn't comprehend the "eq" symbol. Same as > https://gcc.gnu.org/ml/gcc-patches/2018-01/msg01787.html but adding the > change to use "crset 2" instead. > > This one is still regstrapping on BE/LE, trunk/7. If these all complete > successfully, is this okay for trunk and backport?
This looks fine. Okay for trunk if it also works on AIX. For 7 you need RM approval (no objections from me). Thanks! Segher > 2018-01-19 Bill Schmidt <wschm...@linux.vnet.ibm.com> > David Edelsohn <dje....@gmail.com> > > PR target/83946 > * config/rs6000/rs6000.md (*call_indirect_nonlocal_sysv<mode>): > Change "crset eq" to "crset 2". > (*call_value_indirect_nonlocal_sysv<mode>): Likewise. > (*call_indirect_aix<mode>_nospec): Likewise. > (*call_value_indirect_aix<mode>_nospec): Likewise. > (*call_indirect_elfv2<mode>_nospec): Likewise. > (*call_value_indirect_elfv2<mode>_nospec): Likewise. > (*sibcall_nonlocal_sysv<mode>): Change "crset eq" to "crset 2"; > change assembly output from . to $. > (*sibcall_value_nonlocal_sysv<mode>): Likewise. > (indirect_jump<mode>_nospec): Change assembly output from . to $. > (*tablejump<mode>_internal1_nospec): Likewise. > > [gcc/testsuite] > > 2018-01-19 Bill Schmidt <wschm...@linux.vnet.ibm.com> > David Edelsohn <dje....@gmail.com> > > PR target/83946 > * gcc.target/powerpc/safe-indirect-jump-1.c: Change expected > assembly output from "crset eq" to "crset 2". > * gcc.target/powerpc/safe-indirect-jump-2.c: Change expected > assembly output from . to $. > * gcc.target/powerpc/safe-indirect-jump-3.c: Likewise. > * gcc.target/powerpc/safe-indirect-jump-1.c: Change expected > assembly output from "crset eq" to "crset 2". > * gcc.target/powerpc/safe-indirect-jump-8.c: Change expected > assembly output from "crset eq" to "crset 2", and from . to $.
