From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> To: ffmpeg-devel@ffmpeg.org Cc: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Subject: [FFmpeg-devel] [PATCH 3/3] avcodec/fdctdsp: Mark functions as hidden Date: Mon, 24 Jul 2023 15:17:38 +0200 Message-ID: <AS8P250MB0744ABFBF3747B23D3E05A4E8F02A@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> (raw) In-Reply-To: <AS8P250MB074493B91029637EAAB4AACD8F02A@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> Pointers to these functions are used in comparisons. Currently the compiler has to presume the worst for these, namely that the functions are from another DSO and therefore loads their addresses from the GOT (which also entails a relocation entry that is processed at runtime, regardless of whether the code using them is run or not). This changes if these functions are declared as hidden. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> --- libavcodec/fdctdsp.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/fdctdsp.h b/libavcodec/fdctdsp.h index 82edf8b3bb..9dc375322f 100644 --- a/libavcodec/fdctdsp.h +++ b/libavcodec/fdctdsp.h @@ -21,6 +21,7 @@ #include <stdint.h> +#include "libavutil/attributes_internal.h" #include "avcodec.h" typedef struct FDCTDSPContext { @@ -28,6 +29,7 @@ typedef struct FDCTDSPContext { void (*fdct248)(int16_t *block /* align 16 */); } FDCTDSPContext; +FF_VISIBILITY_PUSH_HIDDEN void ff_fdctdsp_init(FDCTDSPContext *c, AVCodecContext *avctx); void ff_fdctdsp_init_ppc(FDCTDSPContext *c, AVCodecContext *avctx, unsigned high_bit_depth); @@ -40,5 +42,6 @@ void ff_jpeg_fdct_islow_8(int16_t *data); void ff_jpeg_fdct_islow_10(int16_t *data); void ff_fdct248_islow_8(int16_t *data); void ff_fdct248_islow_10(int16_t *data); +FF_VISIBILITY_POP_HIDDEN #endif /* AVCODEC_FDCTDSP_H */ -- 2.34.1 _______________________________________________ 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".
prev parent reply other threads:[~2023-07-24 13:17 UTC|newest] Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-07-24 11:59 [FFmpeg-devel] [PATCH] avcodec/(fft|mdct)_fixed_32: Remove fixed-point FFT/MDCT Andreas Rheinhardt 2023-07-24 13:17 ` [FFmpeg-devel] [PATCH 2/3] avcodec/dct: Move fdct function declarations to fdctdsp.h Andreas Rheinhardt 2023-07-29 0:25 ` Andreas Rheinhardt 2023-07-24 13:17 ` Andreas Rheinhardt [this message]
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=AS8P250MB0744ABFBF3747B23D3E05A4E8F02A@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM \ --to=andreas.rheinhardt@outlook.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