In 2.p, two conditions to decrease the P_i are not consistent:
In CG_LOOP::Determine_Best_Unit_Iteration_Interval():
+#ifdef TARG_X8664
+  // now adjust the number of gpr regs as per the ABI
+  P_i--;
+  if (Is_Target_32bit() && Gen_Frame_Pointer)
+    P_i--;
+#endif

In Report_Loop_Info():
+#ifdef TARG_X8664
+    // now adjust the number of gpr regs as per the ABI
+    P_i--;
+    if (Is_Target_32bit() && (PU_src_lang(Get_Current_PU()) ==
PU_CXX_LANG))
+      P_i--;
+#endif

Why are they different?

2011/6/14 Berg, Michael <michael.b...@amd.com>

>  It’s been a while since I sent the request out.  I still need a review
> and I would like to add one more patch file to the set for spec builds.
>
> The third patch is for vmovlps instructions, which ensure spec is ok.  Can
> someone review the set?
>
>
>
> Thx,
>
>
>
> m
>
>
>
> *From:* Berg, Michael
> *Sent:* Saturday, June 04, 2011 3:48 PM
> *To:* open64-devel@lists.sourceforge.net
> *Subject:* [Open64-devel] Gatekeeper code review request for AMD 4.2.5
> merge
>
>
>
> All:  The attached file is a patch containing the cg best fit unrolling
> implementation and is accompanied by a design doc.
>
>         The bulk of the changes are to the cg loop optimizer with some
> changes in lra.  The lra changes are in the first patch file, the bulk of
> the design is in the 2nd.
>
>         The first patch file also contains updates to the interior pointer
> implementation which have lra utility dependencies.
>
>
>
> These changes pass the following:
>
> a.)           No compile time failure for x86 build.
>
> b.)          The gcc regression test suite on x86/Linux with no new
> failures.
>
> c.)           The SPEC2006 test suite at with current AMD 1 copy config at
> both base and peak.
>
>
>
> m
>
>
>
>
> ------------------------------------------------------------------------------
> EditLive Enterprise is the world's most technically advanced content
> authoring tool. Experience the power of Track Changes, Inline Image
> Editing and ensure content is compliant with Accessibility Checking.
> http://p.sf.net/sfu/ephox-dev2dev
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
>


-- 
Regards,
Lai Jian-Xin
------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to