On 4/27/21 11:47 AM, Jakub Jelinek wrote:
On Tue, Apr 27, 2021 at 11:14:00AM -0500, Bill Schmidt wrote:
On 4/27/21 10:57 AM, Jakub Jelinek wrote:
On Tue, Apr 27, 2021 at 10:32:36AM -0500, Bill Schmidt wrote:
2021-03-03 Bill Schmidt <wschm...@linux.ibm.com>
gcc/
* Makefile.in (OUT_FILE_DEPS): New variable.
(out_object_file): Depend on OUT_FILE_DEPS.
Why?
E.g. gcc/config/i386/t-i386 contains:
i386.o: i386-builtin-types.inc
Similarly gcc/config/{arm/t-arm,ia64/t-ia64}
Interesting! Thanks. This was probably an excess of caution on my part.
When I specify something like this, I get a warning message that I am
overriding an automatically generated dependency. If it's okay for me to
ignore that warning, then I can withdraw this patch and simply specify the
dependency directly. Sorry, I should have specified the reason for this in
the patch note.
Can you state what exact warning is it and perhaps can you try to reproduce
that warning with a minimal Makefile example?
I don't remember seeing any such warning on i386 or arm.
Whether the dependency is specified on the goal with the commands to build
it or separately shouldn't really matter.
It turns out that this was me simply misreading the warning. If I
provide the dependency *and* a recipe, I get:
Makefile:2401: warning: overriding recipe for target 'rs6000.o'
/home/wschmidt/newgcc/gcc/config/rs6000/t-rs6000:89: warning: ignoring
old recipe for target 'rs6000.o'
But if I provide just the dependency, the warning doesn't show up, as
expected. So I withdraw this patch, and will change t-rs6000
accordingly instead.
Thanks for the help!
Bill
Jakub
