This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug fortran/79165] 100% compile-time increase for polyhedron aermod by r244581
- From: "rguenther at suse dot de" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Tue, 24 Jan 2017 09:31:29 +0000
- Subject: [Bug fortran/79165] 100% compile-time increase for polyhedron aermod by r244581
- Auto-submitted: auto-generated
- References: <bug-79165-4@http.gcc.gnu.org/bugzilla/>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79165
--- Comment #16 from rguenther at suse dot de <rguenther at suse dot de> ---
On Tue, 24 Jan 2017, m.ostapenko at samsung dot com wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79165
>
> Maxim Ostapenko <m.ostapenko at samsung dot com> changed:
>
> What |Removed |Added
> ----------------------------------------------------------------------------
> CC| |m.ostapenko at samsung dot com
>
> --- Comment #15 from Maxim Ostapenko <m.ostapenko at samsung dot com> ---
> (In reply to David Malcolm from comment #13)
> > What's the purpose of the linemap_add call at line 4761?
>
> I used this to add a source location for TRANSLATION_UNIT_DECL as was discussed
> in pr79061. Dunno, maybe, I've reverted my patch for pr79061 so this change
> isn't present anymore. As Richard pointed out in related mail it would be
> better to do something like this:
>
> + tree tu = build_decl (UNKNOWN_LOCATION, TRANSLATION_UNIT_DECL,
> - name, NULL_TREE);
> + get_identifier (main_input_filename),
> + NULL_TREE);
>
> and use DECL_NAME (translation_unit_decl) later instead of creating new source
> location.
Well, rather pass down get_identifier (main_input_fileame) in the callers.
Richard.