On 11/27/2016 08:50 AM, Thomas Koenig wrote:
Hello world,

here is another, much revised, update of the AVX-specific matmul patch.

The processor-specific switching is now done directly, using the

--- snip ---

This comment not right:

+/* Put exhaustive list of possible architectures here here, ORed together.  */

Performs as expected on my AMD machines. We can still improve peak performance on these by about 7%. To clarify, these chips require -mavx -mprefer-avx128. So what we need to do is sort out which AMD CPUs need this adjustment with AVX registers. (A later patch)

I would like to suggest that rather than matmul_internal.m4 to maybe name this file matmul_base.m4, but not critical.

Need a libgcc person for the changes to the cpuinfo items.

The libgfortran portions look OK.

Jerry

Reply via email to