Follow-up Comment #4, bug #61042 (project make):

Dmitry I'm not sure about this change:


@@ -744,15 +752,9 @@ pattern_search (struct file *file, int archive,
                   /* If this prereq is also explicitly mentioned for FILE,
                      skip all tests below since it must be built no matter
                      which implicit rule we choose. */
-
                   for (expl_d = file->deps; expl_d != 0; expl_d =
expl_d->next)
                     if (streq (dep_name (expl_d), d->name))
                       break;
-                  if (expl_d != 0)
-                    {
-                      (pat++)->name = d->name;
-                      continue;
-                    }
 
                   /* The DEP->changed flag says that this dependency resides
                      in a nonexistent directory.  So we normally can skip


Why are we removing this shortcut here?  It seems like we're paying a cost
everywhere to get better output when debugging is enabled...?  Or is there
some correctness reason to remove this?

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?61042>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/


Reply via email to