This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [buildrobot] [PATCH] r202527: ssa / ssanames restructure broke alpha-linux
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: Jan-Benedict Glaw <jbglaw at lug-owl dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 16 Sep 2013 15:11:16 +0200
- Subject: Re: [buildrobot] [PATCH] r202527: ssa / ssanames restructure broke alpha-linux
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4bRbauJcNx1bM-GY7gau=kqMc3+Gb8AqZ+vTGd0kqw7fA at mail dot gmail dot com> <CAFULd4ZEWA=b2jKdPsksYL0JzcCFtEXam5_g53m=Ws+zdDv6oQ at mail dot gmail dot com> <20130916113655 dot GK24834 at lug-owl dot de> <CAFULd4YOTKzrDnyVLhnVXFK0PkF4iaT_oEOfMmu4m2ANVXbk+g at mail dot gmail dot com> <alpine dot LNX dot 2 dot 00 dot 1309161356140 dot 29729 at zhemvz dot fhfr dot qr>
On Mon, Sep 16, 2013 at 1:57 PM, Richard Biener <rguenther@suse.de> wrote:
>> >> > > My Build Robot[1] found this recent commit to break Alpha:
>> >>
>> >> I think you can remove the tree-flow.h include and you need to update the
>> >> dependencies in gcc/Makefile.in.
>> >>
>> >> However, we still need tree-flow.h, but instead of tree-ssa.h, we can
>> >> include tree-ssanames.h:
>> >>
>> >> I will commit attached incremental patch, once bootstrap finish.
>> >
>> > Please don't forget the dep's in gcc/Makefile.in as mentioned above.
>>
>> Where? I don't see config/alpha.c listed anywhere.
>
> Must be in one of the fragments in config/ or config/alpha. For
> i386 it's config/i386/t-i386
Well, we have a problem here. The dependencies from t-* files are
ignored and the default from generic Makefile.in is used:
>From gcc/Makefile.in:
...
out_file=$(srcdir)/config/@out_file@
out_object_file=@out_object_file@
...
$(out_object_file): $(out_file) $(CONFIG_H) coretypes.h $(TM_H) $(TREE_H) \
$(RTL_H) $(REGS_H) hard-reg-set.h insn-config.h conditions.h \
output.h $(INSN_ATTR_H) $(SYSTEM_H) toplev.h $(DIAGNOSTIC_CORE_H) \
$(TARGET_H) $(LIBFUNCS_H) $(TARGET_DEF_H) $(FUNCTION_H) $(SCHED_INT_H) \
$(TM_P_H) $(EXPR_H) langhooks.h $(GGC_H) $(OPTABS_H) $(REAL_H) \
tm-constrs.h $(GIMPLE_H) $(DF_H) cselib.h $(COMMON_TARGET_H) hw-doloop.h \
regrename.h
$(COMPILER) -c $(ALL_COMPILERFLAGS) $(ALL_CPPFLAGS) \
$(out_file) $(OUTPUT_OPTION)
These are "universal" dependecies used to compile config/${cpu-type}.o
for any target, no matter what their t- fragment says.
Uros.