From: Julian Seward <jseward@acm.org>
Date: Tue, 26 Aug 2014 18:29:26 +0000 (+0000)
Subject: Comment-only change.
X-Git-Tag: svn/VALGRIND_3_10_1^2~38
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4f9e308aa4e86b918fc25fd966ac79a16ede49c7;p=thirdparty%2Fvalgrind.git

Comment-only change.


git-svn-id: svn://svn.valgrind.org/vex/trunk@2934
---

diff --git a/VEX/pub/libvex_ir.h b/VEX/pub/libvex_ir.h
index fffe3b132a..f8cf781d25 100644
--- a/VEX/pub/libvex_ir.h
+++ b/VEX/pub/libvex_ir.h
@@ -1534,6 +1534,13 @@ typedef
       Iop_SarN8x16, Iop_SarN16x8, Iop_SarN32x4, Iop_SarN64x2,
 
       /* VECTOR x VECTOR SHIFT / ROTATE */
+      /* FIXME: I'm pretty sure the ARM32 front/back ends interpret these
+         differently from all other targets.  The intention is that
+         the shift amount (2nd arg) is interpreted as unsigned and
+         only the lowest log2(lane-bits) bits are relevant.  But the
+         ARM32 versions treat the shift amount as an 8 bit signed
+         number.  The ARM32 uses should be replaced by the relevant
+         vector x vector bidirectional shifts instead. */
       Iop_Shl8x16, Iop_Shl16x8, Iop_Shl32x4, Iop_Shl64x2,
       Iop_Shr8x16, Iop_Shr16x8, Iop_Shr32x4, Iop_Shr64x2,
       Iop_Sar8x16, Iop_Sar16x8, Iop_Sar32x4, Iop_Sar64x2,