Please check in Sun On Wed, Dec 1, 2010 at 2:33 PM, Ramanarayanan, Ramshankar <ramshankar.ramanaraya...@amd.com> wrote: > Thanks, yes, I wrote the code in the diff. > > -----Original Message----- > From: Sun Chan [mailto:sun.c...@gmail.com] > Sent: Wednesday, December 01, 2010 11:26 AM > To: Ramanarayanan, Ramshankar > Cc: open64-devel@lists.sourceforge.net > Subject: Re: [Open64-devel] code review request for EXTRACT_BITS simplify > routine > > ok. The change looks fine. This code is completely written by you? > Sun > > On Wed, Dec 1, 2010 at 1:41 PM, Ramanarayanan, Ramshankar > <ramshankar.ramanaraya...@amd.com> wrote: >> Yeah, that is how the functionality is for simp_node functions in >> osprey/common/com/wn_simp_code.h. We are relying on it returning a NULL >> pointer to figure out that a simplification did not happen. The null values >> are passed up the call chains to top level simplifier and in this case it is >> WN_Simplify_Rebuild_Expr_Tree in osprey/common/com/wn_simp.cxx. Please look >> at the calls WN_Simplify_Rebuild_Expr_Tree is making to WN_SimplifyIload, >> WN_SimplifyIntrinsic, WN_SimplifyExp1, WN_SimplifyCvtl, WN_SimplifyExp2, >> WN_SimplifyExp2. >> >> -----Original Message----- >> From: Sun Chan [mailto:sun.c...@gmail.com] >> Sent: Wednesday, December 01, 2010 11:02 AM >> To: Ramanarayanan, Ramshankar >> Cc: open64-devel@lists.sourceforge.net >> Subject: Re: [Open64-devel] code review request for EXTRACT_BITS simplify >> routine >> >> one question, is this ftable thing always return null if not >> successful in simplifying? >> Sun >> >> On Wed, Dec 1, 2010 at 12:42 PM, Ramanarayanan, Ramshankar >> <ramshankar.ramanaraya...@amd.com> wrote: >>> Hi >>> >>> >>> >>> I would like a code review for the attached file >>> "extract_bits_with_simpnode.diff". This is a fix for a recent regression >>> which is seen with virtual destructors. A test case, test_virt_destr.cpp, is >>> attached. >>> >>> >>> >>> The issue is that we are not removing dead code (and unreachable code) in >>> WOPT in case the if-test contains EXTRACT_BITS. I am introducing a simp_node >>> function that bypasses EXTRACT_BITS when it is extracting the last bit of a >>> Boolean constant. This plugs in to the simp_node framework and further >>> simplifications are also possible, in the future. >>> >>> >>> >>> The assembly with and without the patch are test_virt_destr_with_fix.s and >>> test_virt_destr_with_dead_code.s respectively. Please look at the destructor >>> functions: _ZN9TestClassD1Ev, _ZN9TestClassD2Ev for the dead code. >>> >>> >>> >>> Please let me know. >>> >>> >>> >>> Regards, >>> >>> Ram >>> >>> AMD Open Source Compiler Engineering, Bangalore, India >>> >>> ------------------------------------------------------------------------------ >>> Increase Visibility of Your 3D Game App & Earn a Chance To Win $500! >>> Tap into the largest installed PC base & get more eyes on your game by >>> optimizing for Intel(R) Graphics Technology. Get started today with the >>> Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs. >>> http://p.sf.net/sfu/intelisp-dev2dev >>> _______________________________________________ >>> Open64-devel mailing list >>> Open64-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/open64-devel >>> >>> >> >> >> > > >
------------------------------------------------------------------------------ Increase Visibility of Your 3D Game App & Earn a Chance To Win $500! Tap into the largest installed PC base & get more eyes on your game by optimizing for Intel(R) Graphics Technology. Get started today with the Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs. http://p.sf.net/sfu/intelisp-dev2dev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel