git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: What's cooking in git.git (Feb 2016, #07; Thu, 25)

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 26, 2016, 23:35 UTC
Message-ID
<xmqqtwkvhwpd.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<56D0D2DF.1040807@web.de>
Torsten Bögershausen <tboegi@web.de> writes:
Show 29 quoted lines
> On 2016-02-26 19.29, Junio C Hamano wrote:
>> Torsten Bögershausen <tboegi@web.de> writes:
>> 
>>> CC combine-diff.o
>>> combine-diff.c: In function ‘diff_tree_combined’:
>>> combine-diff.c:1391: internal compiler error: Segmentation fault
>>> Please submit a full bug report,
>>> with preprocessed source if appropriate.
>>> See <URL:http://developer.apple.com/bugreporter> for instructions.
>>> make: *** [combine-diff.o] Error 1
>>>
>>> Revert "use st_add and st_mult for allocation size computation" didn't fix it.
>>> I haven't digged deeper yet.
>> 
>> Hmph, I am not quite sure what you meant by "I haven't digged
>> deeper"; I do not think you should be debugging Apple's compiler
>> (unless you have the source to it and that is your job to do so,
>> that is).
>> 
>> In any case, merging the topic to 'master', and reverting that
>> commit will leave this only this change between 'master' and the
>> result.  I do not immediately spot anything trickier in the
>> postimage compared to the preimage that may trickle a compiler bug,
>> but what would I know without the source ;-)
>> 
> Digging means:
> run git bisect and report the commit.
> And this makes the compiler happy:
>   Revert "tree-diff: catch integer overflow in combine_diff_path allocation"
So?

Identifying two versions of the same C file, for one of which the Apple compiler does not give you "internal compiler error" and for the other it does, may help Apple engineers to debug their compiler.

But we cannot and will not revert any of our code only because some vendor compiler is broken. It would be a different story if we were throwing an invalid C at compilers and relying on a bug in GCC that accepts an invalid code, but I do not think that is the case here.

Show 6 quoted lines
>
>     This reverts commit 5b442c4f2723211ce0d862571e88ee206bfd51bf.
>
> Test suite passes,
> ( except t9115, the 2 new test cases "svn.pathnameencoding=cp932".
>   More info about that later)
Previous: Torsten BögershausenNext: Jeff King
Message 5 of 14 in “What's cooking in git.git (Feb 2016, #07; Thu, 25)”
  1. Junio C HamanoFeb 25, 2016
  2. Torsten BögershausenFeb 26, 2016
  3. Junio C HamanoFeb 26, 2016
  4. Torsten BögershausenFeb 26, 2016
  5. Junio C HamanoFeb 26, 2016
  6. Jeff KingFeb 27, 2016
  7. Torsten BögershausenFeb 27, 2016
  8. Torsten BögershausenFeb 27, 2016
  9. Junio C HamanoFeb 28, 2016
  10. 1/1 config.mak.uname: Darwin: Use clang for Mac OS X 10.6tboegi@web.de, Feb 28, 2016
  11. Jeff KingFeb 29, 2016
  12. Eric SunshineFeb 29, 2016
  13. Junio C HamanoFeb 29, 2016
  14. Eric SunshineFeb 29, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.