From: Soft Works <softworkz@hotmail.com> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH v3 0/5] Fix FFmpeg compilation without DCE Date: Sun, 27 Nov 2022 19:23:03 +0000 Message-ID: <DM8P223MB03659A73CF44A3CE5B11D459BA109@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> (raw) In-Reply-To: <CAB0OVGpC9jPCu4=-Ymx3jMLkXQ=k+DCX8=3MOWM83dDRw4j0Qg@mail.gmail.com> > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Carl Eugen Hoyos > Sent: Sunday, November 27, 2022 6:45 PM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v3 0/5] Fix FFmpeg compilation > without DCE > > Am So., 27. Nov. 2022 um 18:16 Uhr schrieb Soft Works > <softworkz@hotmail.com>: > > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Carl Eugen Hoyos > > > Sent: Sunday, November 27, 2022 5:47 PM > > > To: FFmpeg development discussions and patches <ffmpeg- > > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] [PATCH v3 0/5] Fix FFmpeg compilation > > > without DCE > > Please consider fixing your mail client. What did it do wrong? > > > Am So., 27. Nov. 2022 um 17:29 Uhr schrieb Soft Works > > > <softworkz@hotmail.com>: > > > > > > > How do you set this up when you are compiling with MSVC? > > > > > > My configure line to compile with MSVC is: > > > $ src/configure --enable-gpl --toolchain=msvc --host-os=win32 > > > > > > But maybe I misunderstand your question? > > > From what I remember, running configure in MSYS literally > took ages, I don't know if this has improved. Nope. It's still as horribly long as it was, even with everything update to latest.. > > What I meant is the environment. Are you running this from > > an MSYS shell? Or WSL? > > WSL Ah alright. Probably a better choice when all you need is the scripting environment. > > And does running configure output VS project files > > I don't think so. Sorry, I must have confused this with some external script or something. I was asking because when it's about msvc, I'm used to picture someone would be using VS as an IDE. So, yes, the plain msvc compiler doesn't have a problem with DCE. The problem is when creating projects to compile and work with the code in the VS IDE. And what VS does is pre-parsing the code in the background while you're working on it, so you don't really need to compile as you see errors and warning while you write them. As there's no actual linking being done, this procedure cannot "simulate" the DCE process and it can only assume that those symbols that DCE would eliminate are missing symbols. The result is that you cannot work in the IDE because it is full of errors due to this. Likely, this is also the use case of the submitter of this patch. I'm using VS as well, not for final compilation but for writing and debugging as this allows me to work in the way I want to work. Currently, I'm using Matt Oliver's VS project generator, which does a really good job in working around these problems by creating empty definitions for all these loose ends that are missing (as long as a final compilation hasn't determined that they aren't actually needed. But this does also have drawbacks: when changes are made in the ffmpeg base, those definitions may not match anymore and the projects need to be re-generated (which is painful). And the same applies when switching branches, so I usually have multiple sets of VS project/solutions, but these still get outdated and I need to re-create again. For these reasons, I would very much welcome when this situation could be improved. And not only for those but I also had a situation at some time where I wasted a lot of time and got quite desperate on something that turned out to be caused by the code relying on DCE - which wasn't clear to me in that situation. When you know about it, then you can live with it, but when not, then it just adds to the experience for those who are new to ffmpeg. Also, I'm not sure whether that is a reasonable use case for DCE, that it allows you to spare writing a few conditionals in the code. Anyway, I would find it nice if the code would not rely on DCE anymore. Best wishes, softworkz _______________________________________________ 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".
next prev parent reply other threads:[~2022-11-27 19:23 UTC|newest] Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-11-01 21:59 [FFmpeg-devel] [PATCH 0/4] " L. E. Segovia 2022-11-01 21:59 ` [FFmpeg-devel] [PATCH 1/4] all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 L. E. Segovia 2022-11-01 22:28 ` Andreas Rheinhardt 2022-11-03 12:55 ` L. E. Segovia 2022-11-03 14:19 ` Andreas Rheinhardt 2022-11-03 21:59 ` Carl Eugen Hoyos 2022-11-01 21:59 ` [FFmpeg-devel] [PATCH 2/4] all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO L. E. Segovia 2022-11-02 0:38 ` Andreas Rheinhardt 2022-11-01 21:59 ` [FFmpeg-devel] [PATCH 3/4] all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM L. E. Segovia 2022-11-02 0:42 ` Andreas Rheinhardt 2022-11-01 21:59 ` [FFmpeg-devel] [PATCH 4/4] all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM L. E. Segovia 2022-11-02 0:26 ` Andreas Rheinhardt 2022-11-02 1:08 ` [FFmpeg-devel] [PATCH 0/4] Fix FFmpeg compilation without DCE Andreas Rheinhardt 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 0/5] " L. E. Segovia 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 1/5] all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 L. E. Segovia 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 2/5] avcodec/x86/hevcdsp_init: Fix indentation after the ARCH_FOO changes L. E. Segovia 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 3/5] all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO L. E. Segovia 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 4/5] all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM L. E. Segovia 2022-11-03 15:30 ` [FFmpeg-devel] [PATCH v2 5/5] all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM L. E. Segovia 2022-11-03 15:55 ` Lynne 2022-11-14 9:01 ` L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 0/5] Fix FFmpeg compilation without DCE L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 1/5] all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 2/5] avcodec/x86/hevcdsp_init: Fix indentation after the ARCH_FOO changes L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 3/5] all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 4/5] all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM L. E. Segovia 2022-11-03 16:36 ` [FFmpeg-devel] [PATCH v3 5/5] all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM L. E. Segovia 2022-11-07 14:48 ` [FFmpeg-devel] [PATCH v3 0/5] Fix FFmpeg compilation without DCE L. E. Segovia 2022-11-17 22:45 ` L. E. Segovia 2022-11-26 19:17 ` L. E. Segovia 2022-11-27 15:51 ` Carl Eugen Hoyos 2022-11-27 16:29 ` Soft Works 2022-11-27 16:46 ` Carl Eugen Hoyos 2022-11-27 17:16 ` Soft Works 2022-11-27 17:45 ` Carl Eugen Hoyos 2022-11-27 19:23 ` Soft Works [this message] 2022-11-27 17:50 ` Hendrik Leppkes 2022-11-27 18:28 ` Soft Works 2023-07-28 1:37 ` [FFmpeg-devel] [PATCH v4 0/4] Fix MSVC build without optimizations L. E. Segovia 2023-07-28 10:40 ` Reimar Döffinger 2023-07-28 10:48 ` Reimar Döffinger 2023-07-28 10:55 ` Nicolas George 2023-07-28 13:21 ` Matt Oliver 2023-07-29 18:57 ` L. E. Segovia 2023-07-29 19:07 ` [FFmpeg-devel] [PATCH v5 " L. E. Segovia [not found] ` <cover.1690657578.git.amy@amyspark.me> 2023-07-29 19:07 ` [FFmpeg-devel] [PATCH v5 1/4] all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 L. E. Segovia 2023-07-31 18:28 ` Michael Niedermayer 2023-07-29 19:07 ` [FFmpeg-devel] [PATCH v5 2/4] all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO L. E. Segovia 2023-07-29 19:07 ` [FFmpeg-devel] [PATCH v5 3/4] all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM L. E. Segovia 2023-07-29 19:08 ` [FFmpeg-devel] [PATCH v5 4/4] all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM L. E. Segovia [not found] ` <cover.1690508131.git.amy@amyspark.me> 2023-07-28 1:37 ` [FFmpeg-devel] [PATCH v4 1/4] all: Replace if (ARCH_FOO) checks by #if ARCH_FOO, part 2 L. E. Segovia 2023-07-28 1:38 ` [FFmpeg-devel] [PATCH v4 2/4] all: Replace if (CONFIG_FOO) checks by #if CONFIG_FOO L. E. Segovia 2023-07-28 1:38 ` [FFmpeg-devel] [PATCH v4 3/4] all: Guard if (INLINE*) checks with #if HAVE_INLINE_ASM L. E. Segovia 2023-07-28 1:38 ` [FFmpeg-devel] [PATCH v4 4/4] all: Guard if (EXTERNAL*) checks with #if HAVE_X86ASM L. E. Segovia
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=DM8P223MB03659A73CF44A3CE5B11D459BA109@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM \ --to=softworkz@hotmail.com \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git