Oh, very true, I must have missed that -- thank you!
On 2 January 2018 at 14:08, James Almer wrote:
> On 1/2/2018 10:59 AM, Thomas Köppe wrote:
> > Hello,
> >
> > Friendly ping, can I interest you in this patch? It seems like it fixes
> an
> > oversight that would
Hello,
Friendly ping, can I interest you in this patch? It seems like it fixes an
oversight that would be annoying for anyone trying to build FFMPEG for
DJGPP.
Thanks!
On 14 November 2017 at 18:26, Thomas Köppe wrote:
> The macro was added in 43171a2a738f5114768d34a7278e56e5fde714bc, bu
The macro was added in 43171a2a738f5114768d34a7278e56e5fde714bc, but I forgot
to add it to the DJGPP architecture in that change.
---
libavutil/mem.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavutil/mem.h b/libavutil/mem.h
index 9e344bd2c3..7e0b12a8a7 100644
--- a/libavutil/mem.h
+++
Variables used in inline assembly need to be marked with attribute((used)).
Static constants already were, via the define of DECLARE_ASM_CONST.
But DECLARE_ALIGNED does not add this attribute, and some of the variables
defined with it are const only used in inline assembly, and therefore
appeared d
Variables used in inline assembly need to be marked with attribute((used)).
Static constants already were, via the define of DECLARE_ASM_CONST.
But DECLARE_ALIGNED does not add this attribute, and some of the variables
defined with it are const only used in inline assembly, and therefore
appeared d
y vissibility of the use, it is part of
>>> the asm string.
>>>
>>> and comparing this to DECLARE_ALIGNED(), the big difference is
>>> that DECLARE_ALIGNED() is used by plain C code which never should need
>>> "used". So adding "used" t
+Teresa, who drafted the patch initially.
On 31 October 2017 at 15:38, Michael Niedermayer
wrote:
> On Tue, Oct 31, 2017 at 12:16:18PM +0000, Thomas Köppe wrote:
> > Variables used in inline assembly need to be marked with
> attribute((used)).
>
> This should not be the
Variables used in inline assembly need to be marked with attribute((used)).
Static constants already were, via the define of DECLARE_ASM_CONST.
But DECLARE_ALIGNED does not add this attribute, and some of the variables
defined with it are const only used in inline assembly, and therefore
appeared d
30 October 2017 at 20:25, Thomas Köppe wrote:
> (I must have forgotten to rebase first!)
>
> On 30 October 2017 at 20:14, Thomas Köppe wrote:
>
>> It should be under __GNUC__ -- something went wrong with the patch! I'll
>> make a new one. Sorry about that!
>>
>
(I must have forgotten to rebase first!)
On 30 October 2017 at 20:14, Thomas Köppe wrote:
> It should be under __GNUC__ -- something went wrong with the patch! I'll
> make a new one. Sorry about that!
>
> On 30 October 2017 at 19:31, James Almer wrote:
>
>> On 10/30/
It should be under __GNUC__ -- something went wrong with the patch! I'll
make a new one. Sorry about that!
On 30 October 2017 at 19:31, James Almer wrote:
> On 10/30/2017 4:17 PM, Thomas Köppe wrote:
> > Variables used in inline assembly need to be marked with
> attribute((u
Variables used in inline assembly need to be marked with attribute((used)).
Static constants already were, via the define of DECLARE_ASM_CONST.
But DECLARE_ALIGNED does not add this attribute, and some of the variables
defined with it are const only used in inline assembly, and therefore
appeared d
12 matches
Mail list logo
