On 04/08/2010 11:34 AM, mattst88@xxxxxxxxx wrote: > - return __ffs(b0) + ofs; > + unsigned long output; > + asm( > + "cmoveq %0,64,%1 # ofs = (b[0] ? ofs : 64);\n" > + "cmoveq %0,%2,%0 # temp = (b[0] ? b[0] : b[1]);\n" > + "cttz %0,%0 # output = cttz(temp);\n " > + : "=r" (output), "=r" (ofs) > + : "r" (b[1]), "0" (b[0]), "1" (0) > + ); > + return output + ofs; NACK. You need to move that cttz out of the asm as well, to continue to support pre-ev67. Ack if you adjust to use __ffs. r~ -- To unsubscribe from this list: send the line "unsubscribe linux-alpha" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html