Go back to storeAligned / LoadAligned for SkPMFloat <->Sk4f.

src/core/SkPMFloat.h

 #ifndef SkPM_DEFINED
 #define SkPM_DEFINED
 
 #include "SkTypes.h"
 #include "SkColor.h"
 #include "SkColorPriv.h"
 #include "Sk4x.h"
 
 #if SK_CPU_SSE_LEVEL >= SK_CPU_SSE_LEVEL_SSE2
     #include <immintrin.h>
@@ skipping 20 matching lines @@
         fColor[SK_B32_SHIFT / 8] = b;
     }
 
     // Uninitialized.
     SkPMFloat() {}
 
     SkPMFloat(const SkPMFloat& that) { *this = that; }
     SkPMFloat& operator=(const SkPMFloat& that);
 
     // Freely autoconvert between SkPMFloat and Sk4f.  They're always byte-for-byte identical.
-    /*implicit*/ SkPMFloat(const Sk4f& fs) { *(Sk4f*)this = fs; }
-    /*implicit*/ operator Sk4f() const     { return *(const Sk4f*)this;    }
+    /*implicit*/ SkPMFloat(const Sk4f& fs) { fs.storeAligned(fColor); }
+    /*implicit*/ operator Sk4f() const { return Sk4f::LoadAligned(fColor); }
 
     float a() const { return fColor[SK_A32_SHIFT / 8]; }
     float r() const { return fColor[SK_R32_SHIFT / 8]; }
     float g() const { return fColor[SK_G32_SHIFT / 8]; }
     float b() const { return fColor[SK_B32_SHIFT / 8]; }
 
     // get() and clamped() round component values to the nearest integer.
     SkPMColor     get() const;  // May SkASSERT(this->isValid()).  Some implementations may clamp.
     SkPMColor clamped() const;  // Will clamp all values to [0, 255].  Then may assert isValid().
 
@@ skipping 23 matching lines @@
     #include "../opts/SkPMFloat_SSSE3.h"
 #elif SK_CPU_SSE_LEVEL >= SK_CPU_SSE_LEVEL_SSE2
     #include "../opts/SkPMFloat_SSE2.h"
 #elif defined(SK_ARM_HAS_NEON)
     #include "../opts/SkPMFloat_neon.h"
 #else
     #include "../opts/SkPMFloat_none.h"
 #endif
 
 #endif//SkPM_DEFINED