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: [dataflow] PATCH removal of more of flow.c.


James E Wilson wrote:
> On Fri, 2006-07-21 at 14:09, Joern RENNECKE wrote:
>   
>> It's not really my pass; Jim Wilson contributed it before I joined Cygnus.
>>     
>
> I did write the code in the sh_reorg file that uses LOG_LINKS.  This was
> 11 years ago.  It was a reasonable choice at the time, given the
> constraints I was working under (i.e. not enough time and money to do it
> the right way), but it is clearly not reasonable anymore, and it needs
> to be fixed or dropped.  Since this is an optimization, there is no
> serious harm done if we fail to perform it in every case.
>   
I have found that is not really that useful to try find someone to
blame.  What is important is to find the places in the compiler that are
either fragile or broken and then find people who will fix them and then
move on.

The backend is filled with secret invariants that become more
problematic with every change that is made.  My goal is with this
rewrite is to bring the dataflow related ones under control by having
every pass pull it's own weight. 


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