This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: delete unused function longest_path_length

On 08 Feb 2007 07:35:47 -0800, Ian Lance Taylor <iant@google.com> wrote:
"Manuel López-Ibáñez" <lopezibanez@gmail.com> writes:

> :ADDPATCH middle-end:
>
> While fixing PR4076, I found another unused function.
>
> Bootstrapped and regression tested with --enable-languages=all on
> i686-pc-linux-gnu.
>
> Goodbye to it?
>
> 2007-01-31  Manuel Lopez-Ibanez  <manu@gcc.gnu.org>
>
>  * genautomata.c (longest_path_length): Delete unused function.

I'm in favor of deleting this, but please also delete
UNDEFINED_LONGEST_PATH_LENGTH and the longest_path_length field of
struct state.  I'll approve that patch assume it passes bootstrap and
testing.

Please give Vlad (CC'ed) a day or two to comment, maybe he had
something in mind for these.


No problem. I will delete also that, bootstrap + regression test and
post the patch a few days before committing.

Thanks for the review,

Cheers.

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