Commit fbe60463 by Iain Buclaw

d: Generate phony targets for content imported files (PR93038)

This is in addition to the last change which started including them in
the make dependency list.

gcc/d/ChangeLog:

2020-03-22  Iain Buclaw  <ibuclaw@gdcproject.org>

	PR d/93038
	* d-lang.cc (deps_write): Generate phony targets for content imported
	files.

gcc/testsuite/ChangeLog:

2020-03-22  Iain Buclaw  <ibuclaw@gdcproject.org>

	PR d/93038
	* gdc.dg/pr93038b.d: New test.
parent 85e10e4f
2020-03-22 Iain Buclaw <ibuclaw@gdcproject.org> 2020-03-22 Iain Buclaw <ibuclaw@gdcproject.org>
PR d/93038 PR d/93038
* d-lang.cc (deps_write): Generate phony targets for content imported
files.
2020-03-22 Iain Buclaw <ibuclaw@gdcproject.org>
PR d/93038
* d-lang.cc (deps_write): Add content imported files to the make * d-lang.cc (deps_write): Add content imported files to the make
dependency list. dependency list.
......
...@@ -157,26 +157,21 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72) ...@@ -157,26 +157,21 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72)
Modules modlist; Modules modlist;
modlist.push (module); modlist.push (module);
Modules phonylist; vec <const char *> phonylist = vNULL;
const char *str;
unsigned size;
unsigned column = 0; unsigned column = 0;
/* Write out make target module name. */ /* Write out make target module name. */
if (d_option.deps_target) if (d_option.deps_target)
{ {
size = d_option.deps_target->offset; buffer->writestring (d_option.deps_target->extractString ());
str = d_option.deps_target->extractString (); column = d_option.deps_target->offset;
} }
else else
{ {
str = module->objfile->name->str; buffer->writestring (module->objfile->name->str);
size = strlen (str); column = buffer->offset;
} }
buffer->writestring (str);
column = size;
buffer->writestring (":"); buffer->writestring (":");
column++; column++;
...@@ -185,21 +180,25 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72) ...@@ -185,21 +180,25 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72)
{ {
Module *depmod = modlist.pop (); Module *depmod = modlist.pop ();
str = depmod->srcfile->name->str; const char *modstr = depmod->srcfile->name->str;
/* Skip modules that have already been looked at. */ /* Skip modules that have already been looked at. */
if (seen_modules.add (str)) if (seen_modules.add (modstr))
continue; continue;
dependencies.safe_push (str); dependencies.safe_push (modstr);
/* Add to list of phony targets if is not being compile. */ /* Add to list of phony targets if is not being compile. */
if (d_option.deps_phony && !depmod->isRoot ()) if (d_option.deps_phony && !depmod->isRoot ())
phonylist.push (depmod); phonylist.safe_push (modstr);
/* Add imported files to dependency list. */ /* Add imported files to dependency list. */
for (size_t i = 0; i < depmod->contentImportedFiles.dim; i++) for (size_t i = 0; i < depmod->contentImportedFiles.dim; i++)
dependencies.safe_push (depmod->contentImportedFiles[i]); {
const char *impstr = depmod->contentImportedFiles[i];
dependencies.safe_push (impstr);
phonylist.safe_push (impstr);
}
/* Search all imports of the module. */ /* Search all imports of the module. */
for (size_t i = 0; i < depmod->aimports.dim; i++) for (size_t i = 0; i < depmod->aimports.dim; i++)
...@@ -238,8 +237,8 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72) ...@@ -238,8 +237,8 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72)
/* Write out all make dependencies. */ /* Write out all make dependencies. */
for (size_t i = 0; i < dependencies.length (); i++) for (size_t i = 0; i < dependencies.length (); i++)
{ {
str = dependencies[i]; const char *str = dependencies[i];
size = strlen (str); unsigned size = strlen (str);
column += size; column += size;
if (colmax && column > colmax) if (colmax && column > colmax)
...@@ -259,12 +258,10 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72) ...@@ -259,12 +258,10 @@ deps_write (Module *module, OutBuffer *buffer, unsigned colmax = 72)
buffer->writenl (); buffer->writenl ();
/* Write out all phony targets. */ /* Write out all phony targets. */
for (size_t i = 0; i < phonylist.dim; i++) for (size_t i = 0; i < phonylist.length (); i++)
{ {
Module *m = phonylist[i];
buffer->writenl (); buffer->writenl ();
buffer->writestring (m->srcfile->name->str); buffer->writestring (phonylist[i]);
buffer->writestring (":\n"); buffer->writestring (":\n");
} }
} }
......
2020-03-22 Iain Buclaw <ibuclaw@gdcproject.org>
PR d/93038
* gdc.dg/pr93038b.d: New test.
2020-03-22 Iain Sandoe <iain@sandoe.co.uk> 2020-03-22 Iain Sandoe <iain@sandoe.co.uk>
* g++.dg/abi/lambda-vis.C: Amend assembler match * g++.dg/abi/lambda-vis.C: Amend assembler match
......
// https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93038
// { dg-options "-J $srcdir/gdc.dg/fileimports -MMD -MP" }
// { dg-do compile }
// { dg-final { scan-file pr93038b.deps "pr93038b.o: \[^\n\]*/pr93038b.d \[ \\\\\n\]*\[^\n\]*/fileimports/pr93038.txt\n\n\[^\n\]*/fileimports/pr93038.txt:" } }
// { dg-final { file delete pr93038b.deps } }
module pr93038b;
const VERSION = import("pr93038.txt");
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment