[FFmpeg-cvslog] ppc: avutil: Drop a potentially dangerous workaround

Luca Barbato git at videolan.org
Sun May 31 21:31:46 CEST 2015


ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Sun May 10 22:48:30 2015 +0200| [254eb5b6faebb7bcfc3cefc1edced6652fe9d5be] | committer: Luca Barbato

ppc: avutil: Drop a potentially dangerous workaround

The compiler is free to optimize such expressions in any sort of way.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=254eb5b6faebb7bcfc3cefc1edced6652fe9d5be
---

 libavutil/ppc/intreadwrite.h |   14 --------------
 1 file changed, 14 deletions(-)

diff --git a/libavutil/ppc/intreadwrite.h b/libavutil/ppc/intreadwrite.h
index 4471c6a..8f8078d 100644
--- a/libavutil/ppc/intreadwrite.h
+++ b/libavutil/ppc/intreadwrite.h
@@ -24,20 +24,6 @@
 #include <stdint.h>
 #include "config.h"
 
-/*
- * -O0 would compile the packed struct version, which is used by
- * default, in an overly verbose fashion, so we override it here.
- */
-#if HAVE_BIGENDIAN
-#define AV_RB64(p) (*(const uint64_t *)(p))
-#define AV_WB64(p, v) (*(uint64_t *)(p) = (v))
-
-#else
-#define AV_RL64(p) (*(const uint64_t *)(p))
-#define AV_WL64(p, v) (*(uint64_t *)(p) = (v))
-
-#endif
-
 #if HAVE_XFORM_ASM
 
 #if HAVE_BIGENDIAN



More information about the ffmpeg-cvslog mailing list