On Thu, Jul 23, 2015 at 10:21 AM, Segher Boessenkool <seg...@kernel.crashing.org> wrote: > This does one of the TODOs I added: it adds dot forms of the ANDs done > with two machine insns. It uses a new helper function (rs6000_emit_dot_insn) > that probably can be used more often; it is quite general in any case. > > Bootstrapped and tested on powerpc64-linux, using {-m32,-m32/-mpowerpc64, > -m64,-m64/-mlra}; no regressions. Code size on both 32-bit and 64-bit > improves. > > Is this okay for trunk? > > > Segher > > > 2015-07-23 Segher Boessenkool <seg...@kernel.crashing.org> > > PR target/66217 > * config/rs6000/rs6000-protos.h (rs6000_emit_2insn_and): Change > prototype. > * config/rs6000/rs6000.c (rs6000_emit_dot_insn): New function. > (rs6000_emit_2insn_and): Handle dot forms. > * config/rs6000/rs6000.md (and<mode>3): Adjust. > (*and<mode>3_2insn): Remove TODO. Adjust. Add "type" attr. > (*and<mode>3_2insn_dot, *and<mode>3_2insn_dot2): New.
Okay. Thanks, David