Andrew Dunstan <and...@dunslane.net> writes: > 5. most importantly, on my Release/X64 build, I am getting a regression > failure on contrib/seg. regression diffs attached. Until that's fixed > this isn't going anywhere.
I'll bet a nickel that this means MSVC has broken the ABI rules that allowed old-style (version 0) C functions to limp along without changes. seg_same() and the other comparison functions that are misbehaving are declared at the C level to return "bool". I think what's happening is they are now setting only one byte in the return register, or perhaps only the low-order word, leaving the high-order bits as trash. But fmgr_oldstyle is coded as though V0 functions return "char *", and it's going to be putting that whole pointer value into the result Datum, and if the value isn't entirely zero then DatumGetBool will consider it "true". So this theory predicts a lot of intended "false" results will read as "true", which is what your regression diffs show. IIRC, we had intentionally left contrib/seg using mostly V0 calling convention as a canary to let us know when that broke. It's a bit astonishing that it lasted this long, maybe. But it looks like we're going to have to convert at least the bool-returning functions to V1 if we want it to work on VS 2015. Do the other contrib modules all pass? I can't recall if seg was the only one we'd left like this. regards, tom lane -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers