Title: [142336] trunk/Source/WTF
Revision
142336
Author
[email protected]
Date
2013-02-08 17:58:29 -0800 (Fri, 08 Feb 2013)

Log Message

ARM_NEON Inline Assembly for copyLCharsFromUCharSource() inefficient for aligned destinations
https://bugs.webkit.org/show_bug.cgi?id=109335

Reviewed by Filip Pizlo.

Change a "do while" to a "while" so that we don't copy single characters to align the 
destination when it is already aligned.

* wtf/text/ASCIIFastPath.h:
(WTF::copyLCharsFromUCharSource):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (142335 => 142336)


--- trunk/Source/WTF/ChangeLog	2013-02-09 01:48:09 UTC (rev 142335)
+++ trunk/Source/WTF/ChangeLog	2013-02-09 01:58:29 UTC (rev 142336)
@@ -1,3 +1,16 @@
+2013-02-08  Michael Saboff  <[email protected]>
+
+        ARM_NEON Inline Assembly for copyLCharsFromUCharSource() inefficient for aligned destinations
+        https://bugs.webkit.org/show_bug.cgi?id=109335
+
+        Reviewed by Filip Pizlo.
+
+        Change a "do while" to a "while" so that we don't copy single characters to align the 
+        destination when it is already aligned.
+
+        * wtf/text/ASCIIFastPath.h:
+        (WTF::copyLCharsFromUCharSource):
+
 2013-02-08  Jer Noble  <[email protected]>
 
         Bring WebKit up to speed with latest Encrypted Media spec.

Modified: trunk/Source/WTF/wtf/text/ASCIIFastPath.h (142335 => 142336)


--- trunk/Source/WTF/wtf/text/ASCIIFastPath.h	2013-02-09 01:48:09 UTC (rev 142335)
+++ trunk/Source/WTF/wtf/text/ASCIIFastPath.h	2013-02-09 01:58:29 UTC (rev 142336)
@@ -138,9 +138,8 @@
     if (length >= (2 * memoryAccessSize) - 1) {
         // Prefix: align dst on 64 bits.
         const uintptr_t memoryAccessMask = memoryAccessSize - 1;
-        do {
+        while (!isAlignedTo<memoryAccessMask>(destination))
             *destination++ = static_cast<LChar>(*source++);
-        } while (!isAlignedTo<memoryAccessMask>(destination));
 
         // Vector interleaved unpack, we only store the lower 8 bits.
         const uintptr_t lengthLeft = end - destination;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to