Valgrind: r16450 - in /trunk: NEWS none/tests/ppc32/test_isa_2_06_part1.c none/tests/ppc32/test_isa_2_06_part2.c none/tests/ppc32/test_isa_2_06_part3.c

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Valgrind: r16450 - in /trunk: NEWS none/tests/ppc32/test_isa_2_06_part1.c none/tests/ppc32/test_isa_2_06_part2.c none/tests/ppc32/test_isa_2_06_part3.c

svn-2
Author: mjw
Date: Fri Jun 16 10:33:35 2017
New Revision: 16450

Log:
ppc64 doesn't compile test_isa_2_06_partx.c without VSX support

The #ifdef HAS_VSX guard is wrongly placed. It makes the standard
include headers not be used. Causing a build failure. Fix by moving
the #ifdef HAS_VSX after the standard includes.

https://bugs.kde.org/show_bug.cgi?id=381272

Modified:
    trunk/NEWS
    trunk/none/tests/ppc32/test_isa_2_06_part1.c
    trunk/none/tests/ppc32/test_isa_2_06_part2.c
    trunk/none/tests/ppc32/test_isa_2_06_part3.c

Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS (original)
+++ trunk/NEWS Fri Jun 16 10:33:35 2017
@@ -37,6 +37,7 @@
 
 380397  s390x: __GI_strcspn() replacemenet needed
 381162  possible array overrun in VEX register allocator
+381272  ppc64 doesn't compile test_isa_2_06_partx.c without VSX support
 
 
 Release 3.13.0 (15 June 2017)

Modified: trunk/none/tests/ppc32/test_isa_2_06_part1.c
==============================================================================
--- trunk/none/tests/ppc32/test_isa_2_06_part1.c (original)
+++ trunk/none/tests/ppc32/test_isa_2_06_part1.c Fri Jun 16 10:33:35 2017
@@ -20,13 +20,14 @@
  The GNU General Public License is contained in the file COPYING.
  */
 
-#ifdef HAS_VSX
-
 #include <stdio.h>
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 #include <malloc.h>
+
+#ifdef HAS_VSX
+
 #include <altivec.h>
 
 #ifndef __powerpc64__

Modified: trunk/none/tests/ppc32/test_isa_2_06_part2.c
==============================================================================
--- trunk/none/tests/ppc32/test_isa_2_06_part2.c (original)
+++ trunk/none/tests/ppc32/test_isa_2_06_part2.c Fri Jun 16 10:33:35 2017
@@ -20,17 +20,18 @@
  The GNU General Public License is contained in the file COPYING.
  */
 
-#ifdef HAS_VSX
-
 #include <stdio.h>
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 #include <malloc.h>
-#include <altivec.h>
 #include <math.h>
 #include <unistd.h>    // getopt
 
+#ifdef HAS_VSX
+
+#include <altivec.h>
+
 #ifndef __powerpc64__
 typedef uint32_t HWord_t;
 #else

Modified: trunk/none/tests/ppc32/test_isa_2_06_part3.c
==============================================================================
--- trunk/none/tests/ppc32/test_isa_2_06_part3.c (original)
+++ trunk/none/tests/ppc32/test_isa_2_06_part3.c Fri Jun 16 10:33:35 2017
@@ -20,17 +20,18 @@
  The GNU General Public License is contained in the file COPYING.
  */
 
-#ifdef HAS_VSX
-
 #include <stdio.h>
 #include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 #include <malloc.h>
-#include <altivec.h>
 #include <math.h>
 #include <unistd.h>    // getopt
 
+#ifdef HAS_VSX
+
+#include <altivec.h>
+
 #ifndef __powerpc64__
 typedef uint32_t HWord_t;
 #else


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Valgrind-developers mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/valgrind-developers
Loading...