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

Side by Side Diff: src/arm/ic-arm.cc

Issue 2330003: Change the interface of LoadIC on the x64 platform to take its arguments in r... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: '' Created 10 years, 6 months 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | src/ia32/ic-ia32.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 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 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 49
50 // Helper function used from LoadIC/CallIC GenerateNormal. 50 // Helper function used from LoadIC/CallIC GenerateNormal.
51 static void GenerateDictionaryLoad(MacroAssembler* masm, 51 static void GenerateDictionaryLoad(MacroAssembler* masm,
52 Label* miss, 52 Label* miss,
53 Register t0, 53 Register t0,
54 Register t1) { 54 Register t1) {
55 // Register use: 55 // Register use:
56 // 56 //
57 // t0 - used to hold the property dictionary. 57 // t0 - used to hold the property dictionary.
58 // 58 //
59 // t1 - initially the receiver 59 // t1 - initially the receiver.
60 // - used for the index into the property dictionary
61 // - holds the result on exit. 60 // - holds the result on exit.
62 // 61 //
63 // r3 - used as temporary and to hold the capacity of the property 62 // r3 - used as temporary and to hold the capacity of the property
64 // dictionary. 63 // dictionary.
65 // 64 //
66 // r2 - holds the name of the property and is unchanged. 65 // r2 - initially holds the name of the property and is unchanged.
67 // r4 - used as temporary. 66 // r4 - used to hold the index into the property dictionary.
68 67
69 Label done; 68 Label done;
70 69
71 // Check for the absence of an interceptor. 70 // Check for the absence of an interceptor.
72 // Load the map into t0. 71 // Load the map into t0.
73 __ ldr(t0, FieldMemOperand(t1, JSObject::kMapOffset)); 72 __ ldr(t0, FieldMemOperand(t1, JSObject::kMapOffset));
74 73
75 // Bail out if the receiver has a named interceptor. 74 // Bail out if the receiver has a named interceptor.
76 __ ldrb(r3, FieldMemOperand(t0, Map::kBitFieldOffset)); 75 __ ldrb(r3, FieldMemOperand(t0, Map::kBitFieldOffset));
77 __ tst(r3, Operand(1 << Map::kHasNamedInterceptor)); 76 __ tst(r3, Operand(1 << Map::kHasNamedInterceptor));
(...skipping 457 matching lines...) Expand 10 before | Expand all | Expand 10 after
535 534
536 __ bind(&probe); 535 __ bind(&probe);
537 GenerateDictionaryLoad(masm, &miss, r1, r0); 536 GenerateDictionaryLoad(masm, &miss, r1, r0);
538 __ Ret(); 537 __ Ret();
539 538
540 // Global object access: Check access rights. 539 // Global object access: Check access rights.
541 __ bind(&global); 540 __ bind(&global);
542 __ CheckAccessGlobalProxy(r0, r1, &miss); 541 __ CheckAccessGlobalProxy(r0, r1, &miss);
543 __ b(&probe); 542 __ b(&probe);
544 543
545 // Cache miss: Restore receiver from stack and jump to runtime. 544 // Cache miss: Jump to runtime.
546 __ bind(&miss); 545 __ bind(&miss);
547 GenerateMiss(masm); 546 GenerateMiss(masm);
548 } 547 }
549 548
550 549
551 void LoadIC::GenerateMiss(MacroAssembler* masm) { 550 void LoadIC::GenerateMiss(MacroAssembler* masm) {
552 // ----------- S t a t e ------------- 551 // ----------- S t a t e -------------
553 // -- r2 : name 552 // -- r2 : name
554 // -- lr : return address 553 // -- lr : return address
555 // -- r0 : receiver 554 // -- r0 : receiver
(...skipping 1251 matching lines...) Expand 10 before | Expand all | Expand 10 after
1807 GenerateMiss(masm); 1806 GenerateMiss(masm);
1808 } 1807 }
1809 1808
1810 1809
1811 #undef __ 1810 #undef __
1812 1811
1813 1812
1814 } } // namespace v8::internal 1813 } } // namespace v8::internal
1815 1814
1816 #endif // V8_TARGET_ARCH_ARM 1815 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « no previous file | src/ia32/ic-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698