http://codereview.appspot.com/5535046/diff/1/gcc/config/i386/i386.c
File gcc/config/i386/i386.c (right):

http://codereview.appspot.com/5535046/diff/1/gcc/config/i386/i386.c#newcode26032
gcc/config/i386/i386.c:26032: +    M_AMDFAM15,
Maybe better to change 10 to 10H, and 15 to 15H in the name as the
number is hex.

Why not split the enum for family 15h into M_AMDFAM15H_BDVER1 and
.._BVER2?

http://codereview.appspot.com/5535046/diff/1/libgcc/config/i386/i386-cpuinfo.c
File libgcc/config/i386/i386-cpuinfo.c (right):

http://codereview.appspot.com/5535046/diff/1/libgcc/config/i386/i386-cpuinfo.c#newcode102
libgcc/config/i386/i386-cpuinfo.c:102: break;
No family15h model encoding?

http://codereview.appspot.com/5535046/

Reply via email to