RFA: PR 66655: Use COFF/PE weak symbols

Nick Clifton nickc@redhat.com
Tue Jan 5 11:03:00 GMT 2016


Hi Jeff,

> You probably know the capabilities of COFF/PE better than I, so the
> patch itself is fine.

Thanks - committed.

> Is there any way that test can be shoved into our dejagnu testing
> harness?  I think we've got support somewhere for tests which require
> multiple input files.

dg-additional-sources is the command that you are thinking about.


The attached patch adds the test derived from the BZ.  There is one 
small problem - I could not find a way to stop the additional source 
file from being compiled as a test on its own.  When I investigated 
however it seems that is a common problem for all tests that use 
additional source files, so I do not think that it is a big issue.

Tested with no regressions on an x86_64-pc-linux-gnu toolchain.

OK to apply ?

Cheers
   Nick

gcc/testsuite/ChangeLog
2016-01-05  Nick Clifton  <nickc@redhat.com>

	PR target/66655
	* g++.dg/pr66655.C: New test.
	* g++.dg/pr66655_1.C: Additional source file for the test.
	* g++.dg/pr66655.h: Header file for the test.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr66655.test.patch
Type: text/x-patch
Size: 990 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20160105/ea819c62/attachment.bin>


More information about the Gcc-patches mailing list