From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 9AD9149500 for ; Thu, 11 Jul 2024 21:02:55 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id DC3A968DA5F; Fri, 12 Jul 2024 00:02:52 +0300 (EEST) Received: from mail-lj1-f170.google.com (mail-lj1-f170.google.com [209.85.208.170]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6EEC968D982 for ; Fri, 12 Jul 2024 00:02:46 +0300 (EEST) Received: by mail-lj1-f170.google.com with SMTP id 38308e7fff4ca-2ee91d9cb71so14473361fa.0 for ; Thu, 11 Jul 2024 14:02:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=martin-st.20230601.gappssmtp.com; s=20230601; t=1720731765; x=1721336565; darn=ffmpeg.org; h=mime-version:references:message-id:in-reply-to:subject:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=eG7DFQ6XYILxTw1i9hXQbkf5BuCbb43fchjbYHu5y1s=; b=mFdN5GpMYiAmFV9+k5zmFxWaYOPPb91oJMvwG7XPYjOfznlOYsVErncWlbvy7ECL1k S5x/KXm/PmyAy32N/jWovi+2C05vlLOaBJwsuXkq99ffwdXx/FSvSRMf6lGOhgfvbtl0 PZ2fEOd1AEo5swQwnM/yXmm/hyL3xC+4vCAGG5No05n9ONxLOVjZc9+2RqS2me1OE23j 82e5YPtzqU74dl4BGaeE2+RZYoB33ZA5sGVycl6rrA7QNMbNNXbcLDzvT0jyHDAcc058 HV31AGWjDddzpb7Z3V1etKI/RMTAOiq+ZAEYgZeWXPFsbHcZQixuO9JKTz65r5JYkRF3 m/sA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720731765; x=1721336565; h=mime-version:references:message-id:in-reply-to:subject:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=eG7DFQ6XYILxTw1i9hXQbkf5BuCbb43fchjbYHu5y1s=; b=tM+H42Sfn8C1TouwhBRsIM0SLdcx5lxWj2wr2S1JbCFYM+rRl4b0IhP9NgAc4fPb+L r6tFXaPtZbFluxcJZZXeyk7s+81m7v3EjhdUNHX0F04Lr1WLoiVBi4QMCCaLlSJwr745 KCX3XdxWnO1kpu+PbmsvZTUPpaoseWBMI4UY4PrYzgbhapimABCPw0igRp5m546hT5PQ qGRE8DPjmNT6YEM0vBz0kHuGy5z8MfVFlbOlJKJWSKMrbIJlgNqJjCTtWMEWJsNnZdu2 qlomaJa6FH3A3Km07uHivFyebhsv7PL3OjmfZHYb1IqOFP1RwAq/SzNECpvaL/1+7b4N 2GGA== X-Gm-Message-State: AOJu0Yx4PwM/aZg6NOYcG9/a2Ma7LpnWFJ3bAJwTw0QVzUfpEMzhdcCv +aKE/Cy+krwPqDegQBbRLWspPabyY0vb5gFD1jKzfhs05zFujmwtYJyHZxIF+Nn3NZcq79OzhBD e4g== X-Google-Smtp-Source: AGHT+IG31rwuAle17oXHHOLh797qpfOjiS0g/r4WX3LdXbpnXjzkdUKGwygPnlyAWR/TZ37h+Cn7fw== X-Received: by 2002:a2e:9d06:0:b0:2ec:4acf:97dc with SMTP id 38308e7fff4ca-2eeb30bc9eemr64659301fa.11.1720731765541; Thu, 11 Jul 2024 14:02:45 -0700 (PDT) Received: from tunnel335574-pt.tunnel.tserv24.sto1.ipv6.he.net (tunnel335574-pt.tunnel.tserv24.sto1.ipv6.he.net. [2001:470:27:11::2]) by smtp.gmail.com with ESMTPSA id 38308e7fff4ca-2eeb34338b1sm9033211fa.67.2024.07.11.14.02.45 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 11 Jul 2024 14:02:45 -0700 (PDT) Date: Fri, 12 Jul 2024 00:02:44 +0300 (EEST) From: =?ISO-8859-15?Q?Martin_Storsj=F6?= To: FFmpeg development discussions and patches In-Reply-To: <20240711165419.62214-1-jamrial@gmail.com> Message-ID: References: <20240711165419.62214-1-jamrial@gmail.com> MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH] x86/intreadwrite: add missing casts to pointer arguments X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Thu, 11 Jul 2024, James Almer wrote: > Should make strict compilers happy. > Also, make AV_COPY128 use integer operations while at it. > > Signed-off-by: James Almer > --- > libavutil/x86/intreadwrite.h | 15 ++++----------- > 1 file changed, 4 insertions(+), 11 deletions(-) > > diff --git a/libavutil/x86/intreadwrite.h b/libavutil/x86/intreadwrite.h > index d916410e14..65cc6b39a1 100644 > --- a/libavutil/x86/intreadwrite.h > +++ b/libavutil/x86/intreadwrite.h > @@ -23,32 +23,25 @@ > > #include > #include "config.h" > -#if HAVE_INTRINSICS_SSE && defined(__SSE__) > -#include > -#endif If we no longer use HAVE_INTRINSICS_SSE, should we remove the corresponding check in configure too? Thanks, this patch seems to avoid the issue discussed in the other thread. (I'm not familiar enough with these intrinsics to be able to comment meaningfully on the patch itself though.) // Martin _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".