This is broken down into 6 patches.

odc_1288:
- Fix successor list in Do_tail_duplication.

odc_1392:
- skip a corner case that does not need a normalization.
- avoid a seg fault when a SC node does not have a representative block.

odc_1393:
- avoid using a deleted DO_LOOP WHIRL.

odc_1396:
- skip loops that are not flagged LOOP_PRE_DO for normalization.

odc_1398:
- avoid using list iterator to prevent access to deleted SC nodes and etc.

odc_1400:
- Add a heuristic to turn off level 2 loop fusion if there exists a write to 
formal paramters.  This is a workaround for a program bug in CPU2006 gameness.
- Flag pass-by-reference parameters in scalarization to avoid deleting stores 
that appear dead but actually not since the variable is passed by reference


-----Original Message-----
From: Sun Chan [mailto:sun.c...@gmail.com] 
Sent: Monday, May 23, 2011 4:52 PM
To: Ye, Mei
Cc: open64-devel@lists.sourceforge.net
Subject: Re: [Open64-devel] code review: bug fixes for proactive/aggressive 
loop fusions

I understand it is simpler for testing purposes. For later on bug
triage and identification, it would be better if checkins are done one
item at a time (except for hugh source changes). Is it possible to
separate the changes by item?
Sun

On Tue, May 24, 2011 at 7:32 AM, Ye, Mei <mei...@amd.com> wrote:
> This patch contains several bug fixes for proactive/aggressive loop fusions.
>
>
>
> - Fix successor list in "Do_tail_duplication".
>
> - Fix several compilation seg faults for several corner cases like avoid
> using a deleted loop WHIRL;
>
>   skip loops that are not flagged LOOP_PRE_DO for normalization in proactive
> loop fusion; avoid using list iterator to prevent access to deleted SC nodes
> and etc.
>
> - Add a heuristic to turn off level 2 loop fusion if there exists a write to
> formal paramters.  This is a workaround for a program bug in CPU2006
> gameness that was exposed by level 2 loop fusion.
>
> - Flag pass-by-reference parameters in scalarization to avoid deleting
> stores that appear dead but actually not since the variable is passed by
> referenc
>
> ------------------------------------------------------------------------------
> vRanger cuts backup time in half-while increasing security.
> With the market-leading solution for virtual backup and recovery,
> you get blazing-fast, flexible, and affordable data protection.
> Download your free trial now.
> http://p.sf.net/sfu/quest-d2dcopy1
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
>

Attachment: odc_1288
Description: odc_1288

Attachment: odc_1392
Description: odc_1392

Attachment: odc_1393
Description: odc_1393

Attachment: odc_1396
Description: odc_1396

Attachment: odc_1398
Description: odc_1398

Attachment: odc_1400
Description: odc_1400

------------------------------------------------------------------------------
vRanger cuts backup time in half-while increasing security.
With the market-leading solution for virtual backup and recovery, 
you get blazing-fast, flexible, and affordable data protection.
Download your free trial now. 
http://p.sf.net/sfu/quest-d2dcopy1
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to