Title: [124341] trunk/Source/WebCore
Revision
124341
Author
[email protected]
Date
2012-08-01 10:37:16 -0700 (Wed, 01 Aug 2012)

Log Message

Fix gcc 4.7 compile warnings.
https://bugs.webkit.org/show_bug.cgi?id=92797

Patch by Han Shen <[email protected]> on 2012-08-01
Reviewed by Kentaro Hara.

GCC 4.7 gives (more strict) warnings on uninitialized variables,
fixed by providing initilization values.

Tested by gcc 4.7 building. No change in behavior.

* platform/graphics/filters/arm/NEONHelpers.h:
(WebCore::loadRGBA8AsFloat): Add initilizaton value for local variable.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (124340 => 124341)


--- trunk/Source/WebCore/ChangeLog	2012-08-01 16:54:53 UTC (rev 124340)
+++ trunk/Source/WebCore/ChangeLog	2012-08-01 17:37:16 UTC (rev 124341)
@@ -1,3 +1,18 @@
+2012-08-01  Han Shen  <[email protected]>
+
+        Fix gcc 4.7 compile warnings.
+        https://bugs.webkit.org/show_bug.cgi?id=92797
+
+        Reviewed by Kentaro Hara.
+
+        GCC 4.7 gives (more strict) warnings on uninitialized variables,
+        fixed by providing initilization values.
+
+        Tested by gcc 4.7 building. No change in behavior.
+
+        * platform/graphics/filters/arm/NEONHelpers.h:
+        (WebCore::loadRGBA8AsFloat): Add initilizaton value for local variable.
+
 2012-08-01  Ryosuke Niwa  <[email protected]>
 
         REGRESSION(r122498): Assertion failure: m_nodeListCounts is sometimes not zero in the Document destructor

Modified: trunk/Source/WebCore/platform/graphics/filters/arm/NEONHelpers.h (124340 => 124341)


--- trunk/Source/WebCore/platform/graphics/filters/arm/NEONHelpers.h	2012-08-01 16:54:53 UTC (rev 124340)
+++ trunk/Source/WebCore/platform/graphics/filters/arm/NEONHelpers.h	2012-08-01 17:37:16 UTC (rev 124341)
@@ -37,7 +37,8 @@
 
 inline float32x4_t loadRGBA8AsFloat(uint32_t* source)
 {
-    uint32x2_t temporary1 = vset_lane_u32(*source, temporary1, 0);
+    uint32x2_t temporary1 = {0, 0};
+    temporary1 = vset_lane_u32(*source, temporary1, 0);
     uint16x4_t temporary2 = vget_low_u16(vmovl_u8(vreinterpret_u8_u32(temporary1)));
     return vcvtq_f32_u32(vmovl_u16(temporary2));
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to