Skip to content

Commit d1664e7

Browse files
benpeartgitster
authored andcommitted
add: speed up cmd_add() by utilizing read_cache_preload()
During an "add", a call is made to run_diff_files() which calls check_removed() for each index-entry. The preload_index() code distributes some of the costs across multiple threads. Because the files checked are restricted to pathspec, adding individual files makes no measurable impact but on a Windows repo with ~200K files, 'git add .' drops from 6.3 seconds to 3.3 seconds for a 47% savings. Signed-off-by: Ben Peart <benpeart@microsoft.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent d582ea2 commit d1664e7

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

builtin/add.c

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -445,11 +445,6 @@ int cmd_add(int argc, const char **argv, const char *prefix)
445445
return 0;
446446
}
447447

448-
if (read_cache() < 0)
449-
die(_("index file corrupt"));
450-
451-
die_in_unpopulated_submodule(&the_index, prefix);
452-
453448
/*
454449
* Check the "pathspec '%s' did not match any files" block
455450
* below before enabling new magic.
@@ -459,6 +454,10 @@ int cmd_add(int argc, const char **argv, const char *prefix)
459454
PATHSPEC_SYMLINK_LEADING_PATH,
460455
prefix, argv);
461456

457+
if (read_cache_preload(&pathspec) < 0)
458+
die(_("index file corrupt"));
459+
460+
die_in_unpopulated_submodule(&the_index, prefix);
462461
die_path_inside_submodule(&the_index, &pathspec);
463462

464463
if (add_new_files) {

0 commit comments

Comments
 (0)