8066649: 8u backport for 8065618 is incorrect

Summary: The new code in 8u backport should guard the execution of the following code instead of return.
Reviewed-by: iveresov

src/share/vm/opto/ifg.cpp

@@ -541,6 +541,7 @@ uint PhaseChaitin::build_ifg_physical( ResourceArea *a ) {
           if( !n->is_Proj() ||
               // Could also be a flags-projection of a dead ADD or such.
               (_lrg_map.live_range_id(def) && !liveout.member(_lrg_map.live_range_id(def)))) {
+            bool remove = true;
             if (n->is_MachProj()) {
               // Don't remove KILL projections if their "defining" nodes have
               // memory effects (have SCMemProj projection node) -
@@ -553,10 +554,12 @@ uint PhaseChaitin::build_ifg_physical( ResourceArea *a ) {
               for (uint i = 0; i < cnt; i++) {
                 Node* proj = def->raw_out(i);
                 if (proj->Opcode() == Op_SCMemProj) {
-                  return false;
+                  remove = false;
+                  break;
                 }
               }
             }
+            if (remove) {
               block->remove_node(j - 1);
               if (lrgs(r)._def == n) {
                 lrgs(r)._def = 0;
@@ -569,6 +572,7 @@ uint PhaseChaitin::build_ifg_physical( ResourceArea *a ) {
               hrp_index[1]--;
               continue;
             }
+          }
 
           // Fat-projections kill many registers which cannot be used to
           // hold live ranges.