Hi, On Thu, 19 Feb 2026 at 00:26, Nathan Bossart <[email protected]> wrote: > > On Wed, Feb 18, 2026 at 04:38:07PM +0300, Nazir Bilal Yavuz wrote: > > By looking at these results having both is_csv and simd_enabled as an > > argument and sending them as constant boolean arguments help most. > > Thanks for doing these tests. ISTM we might as well get this initial > inlining stuff committed. Thoughts?
nitpick: -static bool CopyReadLineText(CopyFromState cstate, bool is_csv); +static pg_attribute_always_inline bool CopyReadLineText(CopyFromState cstate, + bool is_csv); Do we want to move the new CopyReadLineText() declaration below to group it with the other functions marked pg_attribute_always_inline? Other than that, LGTM. I think it makes sense to separately commit this. -- Regards, Nazir Bilal Yavuz Microsoft
