[meson] add missing pg_attribute_aligned for MSVC in meson build
Hi Andres,
Commit ec3c9cc add pg_attribute_aligned in MSVC[1]/messages/by-id/CAAaqYe-HbtZvR3msoMtk+hYW2S0e0OapzMW8icSMYTMA+mN8Aw@mail.gmail.com,
which was pushed one day before the meson commits,
so meson build missed this feature.
[1]: /messages/by-id/CAAaqYe-HbtZvR3msoMtk+hYW2S0e0OapzMW8icSMYTMA+mN8Aw@mail.gmail.com
--
Regards
Junwang Zhao
Attachments:
0001-meson-add-missing-pg_attribute_aligned-for-MSVC.patchapplication/octet-stream; name=0001-meson-add-missing-pg_attribute_aligned-for-MSVC.patchDownload+2-1
On Fri, Oct 14, 2022 at 10:59:28AM +0800, Junwang Zhao wrote:
Commit ec3c9cc add pg_attribute_aligned in MSVC[1],
which was pushed one day before the meson commits,
so meson build missed this feature.[1]: /messages/by-id/CAAaqYe-HbtZvR3msoMtk+hYW2S0e0OapzMW8icSMYTMA+mN8Aw@mail.gmail.com
Right, thanks! And it is possible to rely on _MSC_VER for that in
this code path.
--
Michael
Hi,
On 2022-10-14 10:59:28 +0800, Junwang Zhao wrote:
Commit ec3c9cc add pg_attribute_aligned in MSVC[1],
which was pushed one day before the meson commits,
so meson build missed this feature.
Good catch. It shouldn't have practical consequences for the moment, given
that msvc doesn't support 128bit integers, but of course we should still be
correct.
Looked through other recent changes to configure and found a few additional
omissions.
See the attached patch fixing those omissions. I'll push it to HEAD once it
has the CI stamp of approval.
Greetings,
Andres Freund