You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by va...@apache.org on 2007/02/09 11:35:47 UTC

svn commit: r505245 - /harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp

Author: varlax
Date: Fri Feb  9 02:35:47 2007
New Revision: 505245

URL: http://svn.apache.org/viewvc?view=rev&rev=505245
Log:
Applied HARMONY-3116 [drlvm][jit][jet] finally block is unreachable when try statement uses while(true) loop
Tested on SUSE10@ia32 and SUSE9@x64

Modified:
    harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp

Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp?view=diff&rev=505245&r1=505244&r2=505245
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/jet/cg_br.cpp Fri Feb  9 02:35:47 2007
@@ -95,7 +95,7 @@
 void Compiler::gen_goto(unsigned target)
 {
     gen_bb_leave(target);
-    if (target < m_pc) {
+    if (target <= m_pc) {
         // Back branch
         gen_prof_be();
         gen_gc_safe_point();
@@ -105,7 +105,7 @@
 
 void Compiler::gen_if(JavaByteCodes opcod, unsigned target)
 {
-    if (target < m_pc) {
+    if (target <= m_pc) {
         // have back branch here
         gen_prof_be();
         gen_gc_safe_point();
@@ -149,7 +149,7 @@
 
 void Compiler::gen_if_icmp(JavaByteCodes opcod, unsigned target)
 {
-    if (target < m_pc) {
+    if (target <= m_pc) {
         // have back branch here
         gen_prof_be();
         gen_gc_safe_point();