On 22 July 2014 12:23, Kyrill Tkachov <kyrylo.tkac...@arm.com> wrote: > Hi all, > > I noticed that we use UNSPEC_CLS for the clrsb optab when we could be using > the RTL code for it. I don't see any reason not to use the RTL code and some > execution tests confirm that the cls instruction implements the expected > clrsb semantics. This patch removes the UNSPEC_CLS definition and usage. > > > Bootstrapped and tested on aarch64-linux. > > Ok for trunk? > > Thanks, > Kyrill > > 2014-07-22 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > * config/aarch64/aarch64.md: Delete UNSPEC_CLS. > (clrsb<mode>2): Use clrsb RTL code instead of UNSPEC_CLS.
Looks Ok. /Marcus
