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