{"thread":{"id":"37098","subject":"745224e0 gcc-4.9 emmintrin.h build error","startedAt":"2014-07-10T19:59:37Z","lastAt":"2014-07-14T06:04:00Z","messageCount":15,"participants":["Tuncer Ayaz","Jeff King","David Turner"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"245706","messageId":"CAOvwQ4hNVvzeCUczi7Qurcycp8HA8KU=u1ntu3fzBwu4fTEzPQ@mail.gmail.com","threadId":"37098","inReplyTo":null,"subject":"745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T19:59:37Z","receivedAt":"2014-07-10T19:59:37Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"The changes in 745224e0 don't seem to build here with gcc-4.9 on\nlinux x64_64. Any idea what's wrong?\n\n CC credential-store.o\nIn file included from /usr/lib/.../xmmintrin.h:31:0,\n from /usr/lib/.../emmintrin.h:31,\n from git-compat-util.h:708,\n from cache.h:4,\n from credential-store.c:1:\n/usr/lib/.../mmintrin.h: In function '_mm_cvtsi32_si64':\n/usr/lib/.../mmintrin.h:64:3: error: can't convert between vector\nvalues of different size\n return (__m64) __builtin_ia32_vec_init_v2si (__i, 0);\n ^\n/usr/lib/.../mmintrin.h: In function '_mm_cvtsi64_si32':\n/usr/lib/.../mmintrin.h:107:10: error: incompatible type for argument\n1 of '__builtin_ia32_vec_ext_v2si'\n return __builtin_ia32_vec_ext_v2si ((__v2si)__i, 0);\n ^\n\n[...]\n\nIn file included from /usr/lib/.../emmintrin.h:31:0,\n from git-compat-util.h:708,\n from cache.h:4,\n from credential-store.c:1:\n/usr/lib/.../xmmintrin.h: In function '_mm_add_ss':\n/usr/lib/.../xmmintrin.h:127:19: error: incompatible type for argument\n1 of '__builtin_ia32_addss'\n return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n ^\n/usr/lib/.../xmmintrin.h:127:3: note: expected '__vector(4) float' but\nargument is of type '__m128'\n return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n ^\n/usr/lib/.../xmmintrin.h:127:19: error: incompatible type for argument\n2 of '__builtin_ia32_addss'\n return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n\n ^\n\n[...]\n\n/usr/lib/.../emmintrin.h:1455:3: error: incompatible type for argument\n2 of '__builtin_ia32_movntpd'\n __builtin_ia32_movntpd (__A, (__v2df)__B);\n ^\n/usr/lib/.../emmintrin.h:1455:3: note: expected '__vector(2) double'\nbut argument is of type '__m128d'\nMakefile:1983: recipe for target 'credential-store.o' failed\nmake: *** [credential-store.o] Error 1\n"},{"id":"245709","messageId":"CAOvwQ4jYkjscTXW5x+Ee=U_Ju0Pjpkkd0wQXiKvoiP3CGhUGXw@mail.gmail.com","threadId":"37098","inReplyTo":"CAOvwQ4hNVvzeCUczi7Qurcycp8HA8KU=u1ntu3fzBwu4fTEzPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T20:06:58Z","receivedAt":"2014-07-10T20:06:58Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Thu, Jul 10, 2014 at 9:59 PM, Tuncer Ayaz wrote:\n> The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> linux x64_64. Any idea what's wrong?\n\ns/x64_64/x86_64/\n\nShould have written amd64 to avoid the typo :).\n\n> CC credential-store.o\n> In file included from /usr/lib/.../xmmintrin.h:31:0,\n> from /usr/lib/.../emmintrin.h:31,\n> from git-compat-util.h:708,\n> from cache.h:4,\n> from credential-store.c:1:\n> /usr/lib/.../mmintrin.h: In function '_mm_cvtsi32_si64':\n> /usr/lib/.../mmintrin.h:64:3: error: can't convert between vector\n> values of different size\n> return (__m64) __builtin_ia32_vec_init_v2si (__i, 0);\n> ^\n> /usr/lib/.../mmintrin.h: In function '_mm_cvtsi64_si32':\n> /usr/lib/.../mmintrin.h:107:10: error: incompatible type for argument\n> 1 of '__builtin_ia32_vec_ext_v2si'\n> return __builtin_ia32_vec_ext_v2si ((__v2si)__i, 0);\n> ^\n>\n> [...]\n>\n> In file included from /usr/lib/.../emmintrin.h:31:0,\n> from git-compat-util.h:708,\n> from cache.h:4,\n> from credential-store.c:1:\n> /usr/lib/.../xmmintrin.h: In function '_mm_add_ss':\n> /usr/lib/.../xmmintrin.h:127:19: error: incompatible type for argument\n> 1 of '__builtin_ia32_addss'\n> return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n> ^\n> /usr/lib/.../xmmintrin.h:127:3: note: expected '__vector(4) float' but\n> argument is of type '__m128'\n> return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n> ^\n> /usr/lib/.../xmmintrin.h:127:19: error: incompatible type for argument\n> 2 of '__builtin_ia32_addss'\n> return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);\n>\n> ^\n>\n> [...]\n>\n> /usr/lib/.../emmintrin.h:1455:3: error: incompatible type for argument\n> 2 of '__builtin_ia32_movntpd'\n> __builtin_ia32_movntpd (__A, (__v2df)__B);\n> ^\n> /usr/lib/.../emmintrin.h:1455:3: note: expected '__vector(2) double'\n> but argument is of type '__m128d'\n> Makefile:1983: recipe for target 'credential-store.o' failed\n> make: *** [credential-store.o] Error 1\n"},{"id":"245711","messageId":"20140710202318.GC15615@sigill.intra.peff.net","threadId":"37098","inReplyTo":"CAOvwQ4hNVvzeCUczi7Qurcycp8HA8KU=u1ntu3fzBwu4fTEzPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2014-07-10T20:23:18Z","receivedAt":"2014-07-10T20:23:18Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Jul 10, 2014 at 09:59:37PM +0200, Tuncer Ayaz wrote:\n\n> The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> linux x64_64. Any idea what's wrong?\n\nWeird. It compiles fine on my x86_64 box (Debian unstable, gcc\n4.9.0-10). Can you tell us more about your environment?\n\n-Peff\n"},{"id":"245712","messageId":"1405024438.3775.3.camel@stross","threadId":"37098","inReplyTo":"CAOvwQ4hNVvzeCUczi7Qurcycp8HA8KU=u1ntu3fzBwu4fTEzPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"David Turner","fromEmail":"dturner@twopensource.com","sentAt":"2014-07-10T20:33:58Z","receivedAt":"2014-07-10T20:33:58Z","isPatch":false,"sender":{"key":"novalis@novalis.org","avatar":"https://avatars.githubusercontent.com/u/77003?v=4"},"body":"On Thu, 2014-07-10 at 21:59 +0200, Tuncer Ayaz wrote:\n> The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> linux x64_64. Any idea what's wrong?\n>\n> CC credential-store.o\n> In file included from /usr/lib/.../xmmintrin.h:31:0,\n\nWhat's in the ...?\n\nBecause if you're using headers from a different version of gcc, that\nmight explain it.\n"},{"id":"245718","messageId":"CAOvwQ4gbaK8x8fQ7DbHpSGmDuvW6bpvFLJRM1fDE21yPYhqOwA@mail.gmail.com","threadId":"37098","inReplyTo":"20140710202318.GC15615@sigill.intra.peff.net","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T20:43:53Z","receivedAt":"2014-07-10T20:43:53Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Thu, Jul 10, 2014 at 10:23 PM, Jeff King wrote:\n> On Thu, Jul 10, 2014 at 09:59:37PM +0200, Tuncer Ayaz wrote:\n>\n> > The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> > linux x64_64. Any idea what's wrong?\n>\n> Weird. It compiles fine on my x86_64 box (Debian unstable, gcc\n> 4.9.0-10). Can you tell us more about your environment?\n\ngcc version 4.9.0 20140604 (prerelease)\n\nI normally use a custom config.mak, but I get the error without\nit too, so it has no direct effect.\n\nLet me know if there's anything to try out for finding the difference.\n"},{"id":"245719","messageId":"CAOvwQ4hq5AZ0ZhB-1etUZfLfJ4X=11_03syH2pgY_fi=FSLAPQ@mail.gmail.com","threadId":"37098","inReplyTo":"1405024438.3775.3.camel@stross","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T20:44:36Z","receivedAt":"2014-07-10T20:44:36Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Thu, Jul 10, 2014 at 10:33 PM, David Turner wrote:\n> On Thu, 2014-07-10 at 21:59 +0200, Tuncer Ayaz wrote:\n> > The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> > linux x64_64. Any idea what's wrong?\n> >\n> > CC credential-store.o\n> > In file included from /usr/lib/.../xmmintrin.h:31:0,\n>\n> What's in the ...?\n>\n> Because if you're using headers from a different version of gcc, that\n> might explain it.\n\n/usr/lib/gcc/x86_64-unknown-linux-gnu/4.9.0/include/emmintrin.h\n"},{"id":"245722","messageId":"1405025616.3775.6.camel@stross","threadId":"37098","inReplyTo":"CAOvwQ4hq5AZ0ZhB-1etUZfLfJ4X=11_03syH2pgY_fi=FSLAPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"David Turner","fromEmail":"dturner@twopensource.com","sentAt":"2014-07-10T20:53:36Z","receivedAt":"2014-07-10T20:53:36Z","isPatch":false,"sender":{"key":"novalis@novalis.org","avatar":"https://avatars.githubusercontent.com/u/77003?v=4"},"body":"On Thu, 2014-07-10 at 22:44 +0200, Tuncer Ayaz wrote:\n> On Thu, Jul 10, 2014 at 10:33 PM, David Turner wrote:\n> > On Thu, 2014-07-10 at 21:59 +0200, Tuncer Ayaz wrote:\n> > > The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> > > linux x64_64. Any idea what's wrong?\n> > >\n> > > CC credential-store.o\n> > > In file included from /usr/lib/.../xmmintrin.h:31:0,\n> >\n> > What's in the ...?\n> >\n> > Because if you're using headers from a different version of gcc, that\n> > might explain it.\n> \n> /usr/lib/gcc/x86_64-unknown-linux-gnu/4.9.0/include/emmintrin.h\n\nThat seems fine to me.\n\nIt looks like the error messages are coming from inside the system's\nheader files (but this is sometimes misleading). If you just try to\ncompile\n\n#include \nint main() { }\n\nwith whatever options you use for git, does that work? If not, I would\nsay that you have a compiler setup problem.\n"},{"id":"245726","messageId":"CAOvwQ4h=uFCN36dQt4uqwJv8393iVhiCh09+71DhhzyWrbCfUA@mail.gmail.com","threadId":"37098","inReplyTo":"1405025616.3775.6.camel@stross","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T21:59:00Z","receivedAt":"2014-07-10T21:59:00Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Thu, Jul 10, 2014 at 10:53 PM, David Turner wrote:\n> On Thu, 2014-07-10 at 22:44 +0200, Tuncer Ayaz wrote:\n> > On Thu, Jul 10, 2014 at 10:33 PM, David Turner wrote:\n> > > On Thu, 2014-07-10 at 21:59 +0200, Tuncer Ayaz wrote:\n> > > > The changes in 745224e0 don't seem to build here with gcc-4.9 on\n> > > > linux x64_64. Any idea what's wrong?\n> > > >\n> > > > CC credential-store.o\n> > > > In file included from /usr/lib/.../xmmintrin.h:31:0,\n> > >\n> > > What's in the ...?\n> > >\n> > > Because if you're using headers from a different version of gcc, that\n> > > might explain it.\n> >\n> > /usr/lib/gcc/x86_64-unknown-linux-gnu/4.9.0/include/emmintrin.h\n>\n> That seems fine to me.\n>\n> It looks like the error messages are coming from inside the system's\n> header files (but this is sometimes misleading). If you just try to\n> compile\n>\n> #include \n> int main() { }\n>\n> with whatever options you use for git, does that work? If not, I would\n> say that you have a compiler setup problem.\n\nThe above test works on the same machine, so I'll\ninvestigate what's going on when building git. Thanks.\n"},{"id":"245727","messageId":"CAOvwQ4in4aufZX2PGkAoX4vWhg_mq+n6Baov_ifGjiyfR_UbPQ@mail.gmail.com","threadId":"37098","inReplyTo":"CAOvwQ4h=uFCN36dQt4uqwJv8393iVhiCh09+71DhhzyWrbCfUA@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-10T22:12:55Z","receivedAt":"2014-07-10T22:12:55Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"Sorry, didn't test properly when I tried with/without config.mak, and\nPROFILE=BUILD was the problem. I had that in config.mak based on\ninformation gathered from INSTALL and Makefile. To be clear, is\nPROFILE=BUILD (still) supported?\n"},{"id":"245731","messageId":"1405033104.3775.10.camel@stross","threadId":"37098","inReplyTo":"CAOvwQ4in4aufZX2PGkAoX4vWhg_mq+n6Baov_ifGjiyfR_UbPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"David Turner","fromEmail":"dturner@twopensource.com","sentAt":"2014-07-10T22:58:24Z","receivedAt":"2014-07-10T22:58:24Z","isPatch":false,"sender":{"key":"novalis@novalis.org","avatar":"https://avatars.githubusercontent.com/u/77003?v=4"},"body":"On Fri, 2014-07-11 at 00:12 +0200, Tuncer Ayaz wrote:\n> Sorry, didn't test properly when I tried with/without config.mak, and\n> PROFILE=BUILD was the problem. I had that in config.mak based on\n> information gathered from INSTALL and Makefile. To be clear, is\n> PROFILE=BUILD (still) supported?\n\nFor what it's worth, the problem seems to depend on the combination of\n-DNO_NOTRETURN=1 and -fprofile-use. So I can trigger the same breakage\nwith this:\n---\n#define NO_NORETURN 1\n#include \"git-compat-util.h\"\n\nint main() {}\n---\ngcc -I. -c -fprofile-use=/tmp foo.c -o foo.o\n\nDo we still need NO_NORETURN? Git seems to build without it under GCC\n4.6.3 (Ubuntu's version). \n"},{"id":"245884","messageId":"20140712045616.GA14459@sigill.intra.peff.net","threadId":"37098","inReplyTo":"CAOvwQ4in4aufZX2PGkAoX4vWhg_mq+n6Baov_ifGjiyfR_UbPQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2014-07-12T04:56:17Z","receivedAt":"2014-07-12T04:56:17Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Jul 11, 2014 at 12:12:55AM +0200, Tuncer Ayaz wrote:\n\n> Sorry, didn't test properly when I tried with/without config.mak, and\n> PROFILE=BUILD was the problem. I had that in config.mak based on\n> information gathered from INSTALL and Makefile. To be clear, is\n> PROFILE=BUILD (still) supported?\n\nI think none of the regular devs uses PROFILE, and it bit-rotted\nsomewhat. Andi Kleen recently posted a series to fix it[1]. I can\nreproduce your problem without that series, but compiling with\nak/profile-feedback-build merged in seems to work OK.\n\n[1] http://thread.gmane.org/gmane.comp.version-control.git/253005\n"},{"id":"245888","messageId":"CAOvwQ4gjfzh4+fNg9-7DecdubzYRW=6t4-4Exq7+mNLAPumb1w@mail.gmail.com","threadId":"37098","inReplyTo":"20140712045616.GA14459@sigill.intra.peff.net","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-12T08:47:03Z","receivedAt":"2014-07-12T08:47:03Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Sat, Jul 12, 2014 at 6:56 AM, Jeff King wrote:\n> On Fri, Jul 11, 2014 at 12:12:55AM +0200, Tuncer Ayaz wrote:\n>\n> > Sorry, didn't test properly when I tried with/without config.mak,\n> > and PROFILE=BUILD was the problem. I had that in config.mak based\n> > on information gathered from INSTALL and Makefile. To be clear, is\n> > PROFILE=BUILD (still) supported?\n>\n> I think none of the regular devs uses PROFILE, and it bit-rotted\n> somewhat. Andi Kleen recently posted a series to fix it[1]. I can\n> reproduce your problem without that series, but compiling with\n> ak/profile-feedback-build merged in seems to work OK.\n>\n> [1] http://thread.gmane.org/gmane.comp.version-control.git/253005\n\nYes, Andi's patch works.\n\nAny idea when ak/profile-feedback-build will land in master?\n"},{"id":"245889","messageId":"CAOvwQ4gEeTS4o6dEUczq+tnS3HePbab-=NpWWA1Z5tkNTjMxfQ@mail.gmail.com","threadId":"37098","inReplyTo":"20140712045616.GA14459@sigill.intra.peff.net","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Tuncer Ayaz","fromEmail":"tuncer.ayaz@gmail.com","sentAt":"2014-07-12T08:53:47Z","receivedAt":"2014-07-12T08:53:47Z","isPatch":false,"sender":{"key":"tuncer.ayaz@gmail.com","avatar":null},"body":"On Sat, Jul 12, 2014 at 6:56 AM, Jeff King wrote:\n\n> I think none of the regular devs uses PROFILE, and it bit-rotted\n\nBy the way, is there no build (CI) server for git.git to regularly\ntest branches on different platforms or at least different build\nconfigs on the same platform?\n"},{"id":"245973","messageId":"20140714060013.GB5593@sigill.intra.peff.net","threadId":"37098","inReplyTo":"CAOvwQ4gjfzh4+fNg9-7DecdubzYRW=6t4-4Exq7+mNLAPumb1w@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2014-07-14T06:00:13Z","receivedAt":"2014-07-14T06:00:13Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sat, Jul 12, 2014 at 10:47:03AM +0200, Tuncer Ayaz wrote:\n\n> Yes, Andi's patch works.\n> \n> Any idea when ak/profile-feedback-build will land in master?\n\nJunio posts updates in his \"What's Cooking\" messages (usually about\ntwice a week). Last Tuesday's listed it as \"will merge to next\". Unless\nanybody finds problems with it, I'd expect it in master within a week or\ntwo, and probably part of v2.1 when it releases.\n\n-Peff\n"},{"id":"245974","messageId":"20140714060400.GC5593@sigill.intra.peff.net","threadId":"37098","inReplyTo":"CAOvwQ4gEeTS4o6dEUczq+tnS3HePbab-=NpWWA1Z5tkNTjMxfQ@mail.gmail.com","subject":"Re: 745224e0 gcc-4.9 emmintrin.h build error","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2014-07-14T06:04:00Z","receivedAt":"2014-07-14T06:04:00Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sat, Jul 12, 2014 at 10:53:47AM +0200, Tuncer Ayaz wrote:\n\n> On Sat, Jul 12, 2014 at 6:56 AM, Jeff King wrote:\n> \n> > I think none of the regular devs uses PROFILE, and it bit-rotted\n> \n> By the way, is there no build (CI) server for git.git to regularly\n> test branches on different platforms or at least different build\n> configs on the same platform?\n\nJunio tests master and next on a few different platforms before pushing\nout the results of topic integration. Other than that, we rely on people\nto report problems (and people testing master or even next on their\nplatforms is very much appreciated, as it means we can fix problems\nbefore they are cemented in a release).\n\n-Peff\n"}]}