NotificationsYou must be signed in to change notification settings
Fork38
Star158

Move all inlined compatible wrappers into header file#57

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

shinderuk merged 1 commit intopostgrespro:masterfromarenadata:inline_compat_functions

Aug 31, 2022

Merged

Move all inlined compatible wrappers into header file#57

shinderuk merged 1 commit intopostgrespro:masterfromarenadata:inline_compat_functions

Aug 31, 2022

Conversation

Copy link

Collaborator

maksm90 commentedAug 17, 2022

Current implementation segregated compatible inline function wrappers into separate .c file that prevented inlining of these functions into caller points in other modules. In current patch all these fuctions are moved to header filecompat.h so that inlining works.

Resolve#45

Move all inlined compatible wrappers into header file

8ac2e6a

Previous implementation segregated compatible inline function wrappersinto separate .c file that prevented inlining of these functions intocaller points in other modules. In current patch all these fuctions with`static inline` specification have been moved to header file compat.h.This emulates behavior of macro expanding in case when compiler performsinlining.Resolvepostgrespro#45