-
-
Notifications
You must be signed in to change notification settings - Fork 134
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix artifacts caused by delayed release of MppPacket data #498
Fix artifacts caused by delayed release of MppPacket data #498
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm fine with this approach but does this extra copy hits performance?
I observed no significant performance difference. But this may change at extremely high bitrates. AMF encoders do the same thing, so this should be fine. |
|
How could aacdec being affected by this? it does not make sense... |
Usually just run it again. Otherwise, Ubuntu may have upgraded mingw and broken it. |
2663532
to
dcd5040
Compare
Is there a specific reason for this? Documenting why it's disabled specifically for this platform seems like a good idea. |
Ubuntu’s recently update breaks GCC LTO in mingw. Our Windows release is using the clang build now so most users won’t be affected, only users specifically choose to use the gcc build will be affected. |
Because of the compiler error mentioned above... We currently have no platforms using this build, and enabling LTO in FFmpeg itself does not magically improve performance like assembly and intrinsics. |
Signed-off-by: nyanmisaka <nst799610810@gmail.com>
Signed-off-by: nyanmisaka <nst799610810@gmail.com>
Signed-off-by: nyanmisaka <nst799610810@gmail.com>
Signed-off-by: nyanmisaka <nst799610810@gmail.com>
dcd5040
to
4e94c66
Compare
Changes
Issues