Re: [PATCH] Location_tize define_label

[Nathan Sidwell <nathan at codesourcery dot com>]

Gabriel Dos Reis wrote:
> Nathan Sidwell <nathan@codesourcery.com> writes:
>
> | --- 2053,2061 ----
> |      Otherwise return 0.  */
> |   
> |   tree
> | ! define_label (location_t location, tree name)
>
> Isn't more efficient to pass a pointer to location instead of copying
> and passing around the structure?  I.e., I would see
>
>    tree
>    define_label (const location_t *locaotion, tree name)
That would make the envtentual line map switchover difficult. And thereby
defeat the whole point of these patches.

nathan

--
Nathan Sidwell    ::   http://www.codesourcery.com   ::     CodeSourcery LLC
         The voices in my head said this was stupid too
nathan@codesourcery.com    ::     http://www.planetfall.pwp.blueyonder.co.uk