Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Side by Side Diff: src/full-codegen.cc

Issue 5699002: RFC: Switch to ast ids (instead of positions) for type feedback. (Closed)
Patch Set: Cleanup Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 727 matching lines...) Expand 10 before | Expand all | Expand 10 after
738 VisitForAccumulatorValue(left); 738 VisitForAccumulatorValue(left);
739 } else { 739 } else {
740 ASSERT(constant == kLeftConstant); 740 ASSERT(constant == kLeftConstant);
741 VisitForAccumulatorValue(right); 741 VisitForAccumulatorValue(right);
742 } 742 }
743 743
744 SetSourcePosition(expr->position()); 744 SetSourcePosition(expr->position());
745 if (ShouldInlineSmiCase(op)) { 745 if (ShouldInlineSmiCase(op)) {
746 EmitInlineSmiBinaryOp(expr, op, mode, left, right, constant); 746 EmitInlineSmiBinaryOp(expr, op, mode, left, right, constant);
747 } else { 747 } else {
748 EmitBinaryOp(op, mode); 748 EmitBinaryOp(expr, op, mode);
749 } 749 }
750 break; 750 break;
751 } 751 }
752 752
753 default: 753 default:
754 UNREACHABLE(); 754 UNREACHABLE();
755 } 755 }
756 } 756 }
757 757
758 758
(...skipping 589 matching lines...) Expand 10 before | Expand all | Expand 10 after
1348 __ Drop(stack_depth); 1348 __ Drop(stack_depth);
1349 __ PopTryHandler(); 1349 __ PopTryHandler();
1350 return 0; 1350 return 0;
1351 } 1351 }
1352 1352
1353 1353
1354 #undef __ 1354 #undef __
1355 1355
1356 1356
1357 } } // namespace v8::internal 1357 } } // namespace v8::internal
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698