This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [vta, trunk?, fortran]


Hi,

Thanks for your contribution.
I think there are some missing parts in your patch.
The trans.h part for example.
Furthermore, as fortran has its own ChangeLog, you don't need to add the
 fortran/ path in the ChangeLog.
Well, I'm not sure about the above, as I don't know how branches are
managed. I can see several ChangeLog.* in the gcc directory.

Mikael

Alexandre Oliva wrote:
> This patch fixes 204 -fcompare-debug failures in GCC fortran
> (including libgomp) testsuites.
> 
> It fixes two separate issues, that each cause divergence in decl uids,
> which in turn lead to optimization differences in compilations with
> and without -g.
> 
> 1. we didn't generate range types for multidimensional arrays when not
>   generating debug info
> 
> 2. we only generated base decls for array descriptors when emitting
>   debug info
> 
> I acknowledge these changes are somewhat wasteful, especially the
> second part, that's most certainly too conservative, but I figured I'm
> not sufficiently familiar with this front-end to figure out a better
> way to do it.
> 
> Suggestions?
> 
> I'm installing this in the VTA branch.
> 
> Is this appropriate for the trunk?
> 
> 
> 
> ------------------------------------------------------------------------
> 
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]