)]}' { "log": [ { "commit": "d2fe10a3a34af171bf1631219cd2d6ff6b7778b5", "tree": "b6b7eb8eba23a5c2723518da99c03bf47b97f58a", "parents": [ "5a3f55ad9519e87c0d3bbddaf3d8a186a887a79b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jan 15 10:20:56 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Feb 17 11:32:15 2014 +0100" }, "message": "Remove blacklist\n\nRemoves the class initialization blacklist and use transaction to detect and\nrevert class initialization attempting to invoke native method. This only\nconcerns class initialization happening at compilation time when generating an\nimage (like boot.art for the system).\n\nIn transactional mode, we log every object\u0027s field assignment and array update.\nTherefore we\u0027re able to abort a transaction to restore values of fields and\narray as they were before the transaction starts. We also log changes to the\nintern string table so we can restore its state prior to transaction start.\n\nSince transactional mode only happens at compilation time, we don\u0027t need to log\nall these changes at runtime. In order to reduce the overhead of testing if\ntransactional mode is on/off, we templatize interfaces of mirror::Object and\nmirror::Array, respectively responsible for setting a field and setting an\narray element.\n\nFor various reasons, we skip some specific fields from transaction:\n- Object\u0027s class and array\u0027s length must remain unchanged so garbage collector\ncan compute object\u0027s size.\n- Immutable fields only set during class loading: list of fields, method,\ndex caches, vtables, ... as all classes have been loaded and verified before a\ntransaction occurs.\n- Object\u0027s monitor for performance reason.\n\nBefore generating the image, we browse the heap to collect objects that need to\nbe written into it. Since the heap may still holds references to unreachable\nobjects due to aborted transactions, we trigger one collection at the end of\nthe class preinitialization phase.\n\nSince the transaction is held by the runtime and all compilation threads share\nthe same runtime, we need to ensure only one compilation thread has exclusive\naccess to the runtime. To workaround this issue, we force class initialization\nphase to run with only one thread. Note this is only done when generating image\nso application compilation is not impacted. This issue will be addressed in a\nseparate CL.\n\nBug: 9676614\nChange-Id: I221910a9183a5ba6c2b99a277f5a5a68bc69b5f9\n" }, { "commit": "d565caf5a5742e2c0b09a1edefac96c8bf354642", "tree": "c72c8bb4f817c0247453fbf3e4ae5252e4c8c65d", "parents": [ "32a6c7f3bd76bbe574675d44b7d8076995690a5b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Feb 16 15:59:00 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Feb 16 16:06:03 2014 -0800" }, "message": "Don\u0027t store this_object in SIRT for static invokes.\n\nI believe this_object is sometimes is garbage for static invokes.\nThis may lead to bad roots being marked if ResolveMethod causes\nsuspension for GC.\n\nBug: 13051126\nChange-Id: I6775b3db63ac5ce37ad423796f16c92645bd7891\n" }, { "commit": "ef7d42fca18c16fbaf103822ad16f23246e2905d", "tree": "c67eea52a349c2ea7f2c3bdda8e73933c05531a8", "parents": [ "822115a225185d2896607eb08d70ce5c7099adef" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jan 06 12:55:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 06 23:20:27 2014 -0800" }, "message": "Object model changes to support 64bit.\n\nModify mirror objects so that references between them use an ObjectReference\nvalue type rather than an Object* so that functionality to compress larger\nreferences can be captured in the ObjectRefererence implementation.\nObjectReferences are 32bit and all other aspects of object layout remain as\nthey are currently.\n\nExpand fields in objects holding pointers so they can hold 64bit pointers. Its\nexpected the size of these will come down by improving where we hold compiler\nmeta-data.\nStub out x86_64 architecture specific runtime implementation.\nModify OutputStream so that reads and writes are of unsigned quantities.\nMake the use of portable or quick code more explicit.\nTemplatize AtomicInteger to support more than just int32_t as a type.\nAdd missing, and fix issues relating to, missing annotalysis information on the\nmutator lock.\nRefactor and share implementations for array copy between System and uses\nelsewhere in the runtime.\nFix numerous 64bit build issues.\n\nChange-Id: I1a5694c251a42c9eff71084dfdd4b51fff716822\n" }, { "commit": "37a9876bf9ff26a451841722a3e3ab9ad4c1e584", "tree": "0370ffd1477301079b8d8bf156a6cbfc0e4e744e", "parents": [ "90ea00c30f5dd1a7c2934417ac5ec7d116ceb93d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 05 12:14:39 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 05 12:24:48 2014 -0800" }, "message": "Add missing SIRT to FindMethodFromCode.\n\nNeeded since ResolveMethod can cause allocations -\u003e GC.\nThis change will hopefully fix the trampoline native crash in the\ndalvik-dev monkeys.\n\nChange-Id: I73aa84a8198142ffb344773372ac80828c225c5c\n" }, { "commit": "89786437f4c0176b35ca0376153dd18ab7df4924", "tree": "c1b233f9e036c14f7a39d722a7a34874253307c1", "parents": [ "1f00671edaaa34578319d0fdaf605600ed539d41" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 31 15:03:55 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 03 18:15:13 2014 +0000" }, "message": "Don\u0027t assume resolved type has the same dex cache.\n\nWhen we resolve a type with a certain DexCache that type\u0027s\nGetDexCache() doesn\u0027t necessarily return the same DexCache.\nThis could have led to the wrong DexFile being used in\naccess checks by the CompilerDriver.\n\nChange-Id: I2c836477f69f142bcbff902207dc0ad83854a398\n" }, { "commit": "bb8f0ab736b61db8f543e433859272e83f96ee9b", "tree": "7b7bac5c0e7d0ac9aba91a40e0fbf761cfacb412", "parents": [ "6e65720d99bd3387b72d528a46291f1ed8184ede" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jan 27 16:50:29 2014 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jan 27 16:50:29 2014 -0800" }, "message": "Embed array class pointers at array allocation sites.\n\nFollowing https://android-review.googlesource.com/#/c/79302, embed\narray class pointers at array allocation sites in the compiled code.\n\nChange-Id: I67a1292466dfbb7f48e746e5060e992dd93525c5\n" }, { "commit": "be1ca55db3362f5b100c4c65da5342fd299520bb", "tree": "b9df6f5562d884698ed15f21764a704bb51e359e", "parents": [ "9d8918fe97c235fdc6eb2c7f2d50a6673ab50329" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Jan 15 11:46:48 2014 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 23 15:29:12 2014 -0800" }, "message": "Use direct class pointers at allocation sites in the compiled code.\n\n- Rather than looking up a class from its type ID (and checking if\n it\u0027s resolved/initialized, resolving/initializing if not), use\n direct class pointers, if possible (boot-code-to-boot-class pointers\n and app-code-to-boot-class pointers.)\n- This results in a 1-2% speedup in Ritz MemAllocTest on Nexus 4.\n- Embedding the object size (along with class pointers) caused a 1-2%\n slowdown in MemAllocTest and isn\u0027t implemented in this change.\n- TODO: do the same for array allocations.\n- TODO: when/if an application gets its own image, implement\n app-code-to-app-class pointers.\n- Fix a -XX:gc bug.\n cf. https://android-review.googlesource.com/79460/\n- Add /tmp/android-data/dalvik-cache to the list of locations to\n remove oat files in clean-oat-host.\n cf. https://android-review.googlesource.com/79550\n- Add back a dropped UNLIKELY in FindMethodFromCode().\n cf. https://android-review.googlesource.com/74205\n\nBug: 9986565\nChange-Id: I590b96bd21f7a7472f88e36752e675547559a5b1\n" }, { "commit": "23a282146042a0d171aec2a415176f5d0621a90c", "tree": "de676891deb5bee6bc76b7cba23c29df662e57c0", "parents": [ "d6c3ae638a47db3970054843d2aa2f0d961cec4c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 09 19:24:37 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 21 17:09:51 2014 +0000" }, "message": "Clean up access checks.\n\nChange-Id: Ia62ba6c8f1d0a9bfbbfde2d7be4c52c0f982b9d2\n" }, { "commit": "e6da9af8dfe0a3e3fbc2be700554f6478380e7b9", "tree": "127a0565fce79f05d82d0ff242fbbffa6e2bc6b3", "parents": [ "977d409b959497eecc44a35cff16115d0f04ccab" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 16 11:54:42 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 08 14:16:12 2014 -0800" }, "message": "Background compaction support.\n\nWhen the process state changes to a state which does not perceives\njank, we copy from the main free-list backed allocation space to\nthe bump pointer space and enable the semispace allocator.\n\nWhen we transition back to foreground, we copy back to a free-list\nbacked space.\n\nCreate a seperate non-moving space which only holds non-movable\nobjects. This enables us to quickly wipe the current alloc space\n(DlMalloc / RosAlloc) when we transition to background.\n\nAdded multiple alloc space support to the sticky mark sweep GC.\n\nAdded a -XX:BackgroundGC option which lets you specify\nwhich GC to use for background apps. Passing in\n-XX:BackgroundGC\u003dSS makes the heap compact the heap for apps which\ndo not perceive jank.\n\nResults:\nSimple background foreground test:\n0. Reboot phone, unlock.\n1. Open browser, click on home.\n2. Open calculator, click on home.\n3. Open calendar, click on home.\n4. Open camera, click on home.\n5. Open clock, click on home.\n6. adb shell dumpsys meminfo\n\nPSS Normal ART:\nSample 1:\n 88468 kB: Dalvik\n 3188 kB: Dalvik Other\nSample 2:\n 81125 kB: Dalvik\n 3080 kB: Dalvik Other\n\nPSS Dalvik:\nTotal PSS by category:\nSample 1:\n 81033 kB: Dalvik\n 27787 kB: Dalvik Other\nSample 2:\n 81901 kB: Dalvik\n 28869 kB: Dalvik Other\n\nPSS ART + Background Compaction:\nSample 1:\n 71014 kB: Dalvik\n 1412 kB: Dalvik Other\nSample 2:\n 73859 kB: Dalvik\n 1400 kB: Dalvik Other\n\nDalvik other reduction can be explained by less deep allocation\nstacks / less live bitmaps / less dirty cards.\n\nTODO improvements: Recycle mem-maps which are unused in the current\nstate. Not hardcode 64 MB capacity of non movable space (avoid\nreturning linear alloc nightmares). Figure out ways to deal with low\nvirtual address memory problems.\n\nBug: 8981901\n\nChange-Id: Ib235d03f45548ffc08a06b8ae57bf5bada49d6f3\n" }, { "commit": "5ddb4104ac605d66693b55b79f26f8b8a5505e63", "tree": "9231f1e54e9a1aaca640c590d5b4f318720e263b", "parents": [ "5a2ced515a456f15dcf194843c024e835eda7dbe" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 07 08:58:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 08 09:15:46 2014 -0800" }, "message": "Remove intialized static storage from dex cache.\n\nThe initialized static storage array is used by compiled code to determine if\nfor a sget/sput class initialization is necessary. The compiled code typically\ndoesn\u0027t require this test as the class is pre-initialized or the class being\naccessed is the same as the current method.\n\nChange-Id: Icbc45e692b3d0ac61e559e69edb6c9b29439e571\n" }, { "commit": "460536209b741bc469f1b0857775449abb2102fb", "tree": "4bc7f636b86ab2531811827d0fd0c33011a7280e", "parents": [ "19031d51ab9edfc3b1996891d654a6b86ca937c0" ], "author": { "name": "Bernhard Rosenkränzer", "email": "Bernhard.Rosenkranzer@linaro.org", "time": "Thu Dec 12 02:15:52 2013 +0100" }, "committer": { "name": "Bernhard Rosenkränzer", "email": "Bernhard.Rosenkranzer@linaro.org", "time": "Thu Dec 12 13:08:03 2013 +0100" }, "message": "Don\u0027t rely on gcc extensions\n\nMake the code more compatible with different compilers.\nclang doesn\u0027t allow extra static qualifiers on template\nspecializations, const qualifiers on function types,\nor inline attributes on lambda functions, and is more\npicky about casting away constness with reinterpret_cast.\n\nThese modifications are compatible with both gcc and\nclang.\n\nChange-Id: I739b10df2780bec537827a13679fd2bcc2cc7188\nSigned-off-by: Bernhard Rosenkränzer \u003cBernhard.Rosenkranzer@linaro.org\u003e\n" }, { "commit": "c528dba35b5faece51ca658fc008b688f8b690ad", "tree": "d95c113a639d07f7e3b88e8a6caa9b5f6e044801", "parents": [ "76f552307ee980221626d1dff0ada56c989d24ca" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 26 12:00:11 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 10 16:00:58 2013 -0800" }, "message": "Enable moving classes.\n\nSlight reduction in Zygote size, memory savings are in the noise.\nBefore: Zygote size: 8739224\nAfter: Zygote size: 8733568\n\nFixed a bug where we didn\u0027t set the concurrent start bytes after\nswitching the allocator from bump pointer to ROSAlloc in the\nzygote. This caused excessive memory usage.\n\nAdded the method verifiers as roots to fix an issue caused by\nRegTypes holding a Class*.\n\nAdded logic to clear card table in the SemiSpace collector, this\nreduces DalvikOther from ~2400k -\u003e ~1760k when using the SemiSpace\ncollector.\n\nAdded a missing lock to the timing loggers which caused a rare\none time crash in std::set.\n\nBug: 11771255\nBug: 8499494\nBug: 10802951\n\nChange-Id: I99d2b528cd51c1c5ed7012e3220b3aefded680ae\n" }, { "commit": "cbb2d20bea2861f244da2e2318d8c088300a3710", "tree": "9735d496716cf165ea0ee2d7e2f62d723ffc7734", "parents": [ "d31fb9718a6180304cd951619dc36be8e090a641" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 14 17:45:16 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 20 11:14:11 2013 -0800" }, "message": "Refactor allocation entrypoints.\n\nAdds support for switching entrypoints during runtime. Enables\naddition of new allocators with out requiring significant copy\npaste. Slight speedup on ritzperf probably due to more inlining.\n\nTODO: Ensuring that the entire allocation path is inlined so\nthat the switch statement in the allocation code is optimized\nout.\n\nRosalloc measurements:\n4583\n4453\n4439\n4434\n4751\n\nAfter change:\n4184\n4287\n4131\n4335\n4097\n\nChange-Id: I1352a3cbcdf6dae93921582726324d91312df5c9\n" }, { "commit": "590fee9e8972f872301c2d16a575d579ee564bee", "tree": "b02db45c72f1911ec896b93379ada0276aea3199", "parents": [ "5b70680b8df6d8fa95bb8e1070d0107f3d388940" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 13 13:46:47 2013 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 11 15:34:27 2013 -0800" }, "message": "Compacting collector.\n\nThe compacting collector is currently similar to semispace. It works by\ncopying objects back and forth between two bump pointer spaces. There\nare types of objects which are \"non-movable\" due to current runtime\nlimitations. These are Classes, Methods, and Fields.\n\nBump pointer spaces are a new type of continuous alloc space which have\nno lock in the allocation code path. When you allocate from these it uses\natomic operations to increase an index. Traversing the objects in the bump\npointer space relies on Object::SizeOf matching the allocated size exactly.\n\nRuntime changes:\nJNI::GetArrayElements returns copies objects if you attempt to get the\nbacking data of a movable array. For GetArrayElementsCritical, we return\ndirect backing storage for any types of arrays, but temporarily disable\nthe GC until the critical region is completed.\n\nAdded a new runtime call called VisitObjects, this is used in place of\nthe old pattern which was flushing the allocation stack and walking\nthe bitmaps.\n\nChanged image writer to be compaction safe and use object monitor word\nfor forwarding addresses.\n\nAdded a bunch of added SIRTs to ClassLinker, MethodLinker, etc..\n\nTODO: Enable switching allocators, compacting on background, etc..\n\nBug: 8981901\n\nChange-Id: I3c886fd322a6eef2b99388d19a765042ec26ab99\n" }, { "commit": "88474b416eb257078e590bf9bc7957cee604a186", "tree": "7c59aa370bec9b0f2d37cb7a96d3b2effb3d92ce", "parents": [ "9780099e445884d8bc9444c8c1261b02d80a26c7" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Oct 23 16:24:40 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Oct 29 12:01:28 2013 -0700" }, "message": "Implement Interface Method Tables (IMT).\n\nChange-Id: Idf7fe85e1293453a8ad862ff2380dcd5db4e3a39\n" }, { "commit": "d4beb6bc2b42b176c6d04fdd91d6c758e542c7c2", "tree": "083f8c75e44105e98688eff3e206e5a7fd188912", "parents": [ "57e6d8a99058e5c74d5244b68a5f4d53526fa108" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Oct 02 17:07:20 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Oct 28 18:12:22 2013 +0100" }, "message": "Inline field and method resolution.\n\nAccording to profiling results, field and method resolutions are hot points\nduring interpreter execution. This CL attempts to speed up these resolutions.\n\nForces aggressive inlining of FindFieldFromCode and FindMethodFromCode. This\nallows to reduce the overhead of access check code when the interpreter runs\nwithout these checks. Templatize these functions to optimize inlining and their\ncallers.\n\nAlso spread the use of C++11 \"nullptr\" in place of \"NULL\" in field access and\ninvoke helpers.\n\nChange-Id: Ic1a69834d8975b2cddcddaae32f08a7de146a951\n" }, { "commit": "241b5de2d3cf06868ac31f1153aa0b32ddb07b20", "tree": "6161a90ea504c3907d171f16ca7d94817bbf8b9a", "parents": [ "252a00273bc945ebe4a1d77d82d085fdff2ae769" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 09 17:58:57 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 10 17:26:04 2013 -0700" }, "message": "Clinits may not have the kAccConstructor flag.\n\nBug: 11157540\nSet the clinit access flag when we load the method and warn about badly\nformed access flags.\n\nChange-Id: I515c692095051f84f98510722ab764591185918e\n" }, { "commit": "d9c4fc94fa618617f94e1de9af5f034549100753", "tree": "1305efbbc3d4bc306c0947bb6d4b01553667f98e", "parents": [ "7ef126ce0593929bcf8fb73d8b1119ce3b95b3f2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 01 19:45:43 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 02 09:31:55 2013 -0700" }, "message": "Inflate contended lock word by suspending owner.\n\nBug 6961405.\nDon\u0027t inflate monitors for Notify and NotifyAll.\nTidy lock word, handle recursive lock case alongside unlocked case and move\nassembly out of line (except for ARM quick). Also handle null in out-of-line\nassembly as the test is quick and the enter/exit code is already a safepoint.\nTo gain ownership of a monitor on behalf of another thread, monitor contenders\nmust not hold the monitor_lock_, so they wait on a condition variable.\nReduce size of per mutex contention log.\nBe consistent in calling thin lock thread ids just thread ids.\nFix potential thread death races caused by the use of FindThreadByThreadId,\nmake it invariant that returned threads are either self or suspended now.\n\nCode size reduction on ARM boot.oat 0.2%.\nOld nexus 7 speedup 0.25%, new nexus 7 speedup 1.4%, nexus 10 speedup 2.24%,\nnexus 4 speedup 2.09% on DeltaBlue.\n\nChange-Id: Id52558b914f160d9c8578fdd7fc8199a9598576a\n" }, { "commit": "3b4c18933c24b8a33f38573c2ebcdb9aa16efeb5", "tree": "5298ccd9c1f1f6b329c0cb6cefac6a8df43dd633", "parents": [ "f7e090ebcded6d6693894c018d89c4add79253ff" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Sep 12 21:33:12 2013 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Sep 25 20:28:49 2013 -0700" }, "message": "Split the allocation path into \u0027instrumented\u0027 and \u0027uninstrumented\u0027\nones.\n\nThe instrumented path is equivalent to the existing allocation path\nthat checks for three instrumentation mechanisms (the debugger\nallocation tracking, the runtime allocation stats collection, and\nvalgrind) for every allocation. The uinstrumented path does not\nperform these checks. We use the uninstrumented path by default and\nenable the instrumented path only when any of the three mechanisms is\nenabled. The uninstrumented version of Heap::AllocObject() is inlined.\n\nThis change improves the Ritz MemAllocTest by ~4% on Nexus 4 and ~3%\non Host/x86.\n\nBug: 9986565\nChange-Id: I3e68dfff6789d77bbdcea98457b694e1b5fcef5f\n" }, { "commit": "450dcb56ecbf6f729401e753f0a27e4170177ddd", "tree": "f596d57c5f6fbdc90b3dc68a2008e38ee48e2882", "parents": [ "810b1d704f2db0d935bf5dddae3545f79cabd435" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 20 17:36:02 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 20 18:47:39 2013 -0700" }, "message": "Improve float to integral conversion.\n\nChange-Id: I1597083cb2c04084ce825fe2e3c753fde8309cd8\n" }, { "commit": "bd136a29f08486525d6abc7d0a0006ce5b4011c1", "tree": "71a61d4545e7fa871d2cafee23dd5e3beb6619ee", "parents": [ "7d70a7932f0ba09eb01a93caab060aef1403d4e6" ], "author": { "name": "Dragos Sbirlea", "email": "dragoss@google.com", "time": "Tue Aug 13 18:07:04 2013 -0700" }, "committer": { "name": "Dragos Sbirlea", "email": "dragoss@google.com", "time": "Thu Aug 15 15:10:55 2013 -0700" }, "message": "Get PORTABLE + SMALL on x86.\n\nImplemented the portable resolution trampoline and\nthe portable to interpreter bridge.\nAlso work on integrating SEA_IR in the PORTABLE+SMALL framework.\nRefactor some naming and correct indenting.\n\nChange-Id: Ibd97da5e5b6f5148274c9bff368e3654b661ef51\n" }, { "commit": "ea46f950e7a51585db293cd7f047de190a482414", "tree": "9dddc8073547a2dcb58a19e1728932a89cb149c3", "parents": [ "5e3572709b5a5d59957f835db4f73760ecef08da" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Jul 30 01:26:50 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 18:09:46 2013 -0700" }, "message": "Refactor java.lang.reflect implementation\n\nCherry-picked from commit ed41d5c44299ec5d44b8514f6e17f802f48094d1.\n\nMove to ArtMethod/Field instead of AbstractMethod/Field and have\njava.lang.reflect APIs delegate to ArtMethod/ArtField.\n\nBug: 10014286.\n\nChange-Id: Iafc1d8c5b62562c9af8fb9fd8c5e1d61270536e7\n" }, { "commit": "848871b4d8481229c32e0d048a9856e5a9a17ef9", "tree": "4be4602e3becc899f76a29a87618801458fe04b9", "parents": [ "423fb4d70f2ac36bf9f630146b4150771a8e7e76" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Aug 05 10:56:33 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 07 22:17:00 2013 -0700" }, "message": "Entry point clean up.\n\nCreate set of entry points needed for image methods to avoid fix-up at load time:\n - interpreter - bridge to interpreter, bridge to compiled code\n - jni - dlsym lookup\n - quick - resolution and bridge to interpreter\n - portable - resolution and bridge to interpreter\n\nFix JNI work around to use JNI work around argument rewriting code that\u0027d been\naccidentally disabled.\nRemove abstact method error stub, use interpreter bridge instead.\nConsolidate trampoline (previously stub) generation in generic helper.\nSimplify trampolines to jump directly into assembly code, keeps stack crawlable.\nDex: replace use of int with ThreadOffset for values that are thread offsets.\nTidy entry point routines between interpreter, jni, quick and portable.\n\nChange-Id: I52a7c2bbb1b7e0ff8a3c3100b774212309d0828e\n" }, { "commit": "7655f29fabc0a12765de828914a18314382e5a35", "tree": "219f2df20cf7f053a73a345ae20e131a73759c79", "parents": [ "1a8080d4c2e1772cfcc5e9d6587bc63bba3945d9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jul 29 11:07:13 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 30 16:00:24 2013 -0700" }, "message": "Portable refactorings.\n\nSeparate quick from portable entrypoints.\nMove architectural dependencies into arch.\n\nChange-Id: I9adbc0a9782e2959fdc3308215f01e3107632b7c\n" }, { "commit": "02c8cc6d1312a2b55533f02f6369dc7c94672f90", "tree": "4e4a268e8cb82dcd95f8e29a8169b0396e20b0cc", "parents": [ "6f485c62b9cfce3ab71020c646ab9f48d9d29d6d" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 15:54:44 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 16:02:33 2013 -0700" }, "message": "Fixing cpplint whitespace/blank_line, whitespace/end_of_line, whitespace/labels, whitespace/semicolon issues\n\nChange-Id: Ide4f8ea608338b3fed528de7582cfeb2011997b6\n" }, { "commit": "f69863b3039fc621ff4250e262d2a024d5e79ec8", "tree": "b6d40c4019de2155f8753f83b4750bd3a3514d7a", "parents": [ "93ba893c20532990a430741e0a97212900094e8c" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:53:13 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 23:23:12 2013 -0700" }, "message": "Fix cpplint whitespace/newline issues\n\nChange-Id: Ie2049d9f667339e41f36c4f5d09f0d10d8d2c762\n" }, { "commit": "fc0e3219edc9a5bf81b166e82fd5db2796eb6a0d", "tree": "5fc7f5b941724a62f8e3411df09fae431ff5e3cf", "parents": [ "56d947fbc9bc2992e2f93112fafb73e50d2aaa7a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 14:40:12 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 16:49:05 2013 -0700" }, "message": "Fix multiple inclusion guards to match new pathnames\n\nChange-Id: Id7735be1d75bc315733b1773fba45c1deb8ace43\n" }, { "commit": "7940e44f4517de5e2634a7e07d58d0fb26160513", "tree": "ac90242d96229a6942f6e24ab137bc1f8f2e0025", "parents": [ "5cd9e3b122f276f610980cbaf0d2ad6ed4cd9088" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 13:46:57 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 17:49:01 2013 -0700" }, "message": "Create separate Android.mk for main build targets\n\nThe runtime, compiler, dex2oat, and oatdump now are in seperate trees\nto prevent dependency creep. They can now be individually built\nwithout rebuilding the rest of the art projects. dalvikvm and jdwpspy\nwere already this way. Builds in the art directory should behave as\nbefore, building everything including tests.\n\nChange-Id: Ic6b1151e5ed0f823c3dd301afd2b13eb2d8feb81\n" }, { "commit": "233ea8e084a95ad2a3af746dddbadb155db6a814", "tree": "be5eeee19b2274e1b199a230e9297f277b47d155", "parents": [ "8afe6e0d8c4151ac74afc4ebe0dab85550821243" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Jun 06 11:57:09 2013 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Jun 13 17:15:16 2013 +0200" }, "message": "No access check support.\n\nThis CL adds support to disable access check when a method is preverified (at\ncompilation time) and we know we don\u0027t need to do any access check.\n\nThe interpreter has now two modes of execution: with or without access check.\nThis is realized by using a template function.\n\nA new runtime access flag kAccPreverified is added onto each method belonging\nto a preverified class. If this flag is set, we enter the interpreter in \"no\naccess check\" mode. Otherwise, we enter the interpreter in \"with access check\"\nmode.\n\nChange-Id: Ic34163421d5b0aca3d1bce22ef7c095dcf465a18\n" }, { "commit": "0aba0ba155bef7346bde19e53581200b89ae8a7a", "tree": "6590f8a50bde7c30b57f1bb0ae67c6d88cea8116", "parents": [ "515661b14b60ab9684efa1ab1d5124934406094e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jun 03 14:49:28 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Jun 07 14:43:24 2013 -0700" }, "message": "Created compiled stubs in image.\n\nSaves class linker from having to set code pointers when loading\nfrom an image. Added stubs for quick and portable resolution\ntrampolines, and interpreter-to-interpreter and interpreter-to-quick\nentry points. Also added sizing stats output for oat writer.\n\nChange-Id: I3905fae81047742c23d1cf0ca001db798db971b1\n" }, { "commit": "fa46d3ee8cba2985c5c8bdfe47d15daf0613ebdc", "tree": "2743be5358aef17e9ae21b0626409feeea919771", "parents": [ "9cec247849c248f6c45a14eae5414b69eb8fc67d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed May 15 00:16:04 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed May 15 11:03:44 2013 -0700" }, "message": "Interpreter tweaks\n\nIncrease the amount of inlining and hot function hints in the interpreter\nto encourage the hot Execute function to be faster.\nPerformance is 3x slower than Dalvik+JIT on FibonacciFast and similar\nmicrobenchmarks.\n\nChange-Id: I2b1a0c7545f86036b9b1b5ccac881d06292356d8\n" }, { "commit": "79fe539587d4c09244172d0168eeed0ec9770466", "tree": "4fb8fbd03727a74acb4dfde7e969adf32ee32750", "parents": [ "09b55e1df3b0bcecee25882e45e882887cc8654f" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Apr 24 18:41:58 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Apr 26 14:50:52 2013 -0700" }, "message": "Removing remaining compiled stubs from image.\n\nAbstract method error stub and jni dlsym lookup stubs are gone.\nAfter this change, the image no longer needs to be executable.\n\nChange-Id: Ic75d72bf7e76e3b8ecc596e82af68ab592dde15e\n" }, { "commit": "58df32711162823647356afe25ae2cde04133563", "tree": "df1b2b43b90f4861b475f1452b175353a56ec3d7", "parents": [ "ede1807e3481a6937416964bb7556364f2d428e1" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 22 15:28:53 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 23 16:34:16 2013 -0700" }, "message": "Removed compiled resolution trampoline.\n\nCreated assembly versions of resolution trampoline for portable and\nquick. Removed compiled version from the image.\n\nChange-Id: I183e110c4e102fb45ce6e7f4e855f8bed7986251\n" }, { "commit": "62d6c772205b8859f0ebf7ad105402ec4c3e2e01", "tree": "e2f2ba6d71ed5a39c9f6909e3f7c08e998053315", "parents": [ "c9b17c7ee96cd04fac9048aab624ed554fe260bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 08:32:07 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 08 14:24:13 2013 -0700" }, "message": "Interpreter entries and instrumentation as a listener.\n\nMake the instrumentation responsible for whether we want method entry/exit\nstubs, and allow it to use interpreter entry stubs when instruction by\ninstruction instrumentation is required. Improve deoptimization so more JDWP\ntest cases are passing.\n\nRefactor exception debug posting, in particular improve reporting in the\ninterpreter. Improve class linker exception throwing so that broken dex files\nare more likely to be reported. Fixes the performance issue Bug: 8410519.\n\nFix some error reporting lock level errors for the large object space. Make\nfast object verification faster.\n\nAdd some debug mode robustness to finding dex PCs in GC maps.\n\nAdd printf attributes to JniAbortF and fix errors.\n\nExpand run-test 044 to test return behaviors and fix issues with not throwing\nappropriate exceptions for proxies.\n\nEnsure causes are reported with a class linker NoClassDefFoundError and JNI\nNoSuchFieldError.\n\nRemove unused debugMe and updateDebuggerFromCode.\n\nThere\u0027s a minor sizing tweak to the arg array builder, and an extra reference\narray check in the interpreter.\n\nSome clean-up of trace code.\n\nFix reg type cache destructor if it is called after the reg type cache is\nshutdown (as is the case in oatdump).\n\nChange-Id: I6519c7b35df77f978d011999354c864f4918e8ce\n" }, { "commit": "5fa60c3db4208df407113b5a69d295a9c93d53b1", "tree": "b56704a7f97cb524c2cf269700fabcae25bf0f08", "parents": [ "a966e18b402b0568fc556c38d7ef2e7e8df943a2" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Apr 04 17:57:01 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Apr 05 11:01:06 2013 -0700" }, "message": "Remove use of compiled proxy stub from portable.\n\nARM implementation is tested, but x86 and MIPS are not.\n\nChange-Id: I497d650c55fe7fd4ea24d283c8d8ee99b1456610\n" }, { "commit": "8dbb708c7dc05c786329eb5c3fff3194ab6472ac", "tree": "1374ac8244bb13612e027de23516ecddfb2bba3c", "parents": [ "cc07edb6bb274593c687188c99015c30dd73acae" ], "author": { "name": "Logan Chien", "email": "loganchien@google.com", "time": "Fri Jan 25 20:31:17 2013 +0800" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Feb 06 19:14:06 2013 -0800" }, "message": "Rename run-time functions to distinguish art_quick_[name] vs art_portable_[name].\n\nWe have two different sets of run-time functions:\n(1) the one used by quick compiler, and\n(2) the one used by portable compiler.\n\nPreviosuly, the one used by quick compiler is placed in ``extern\n\"C\"\u0027\u0027, and the one used by portable compiler is placed in ``namespace\nart\u0027\u0027. That\u0027s why we can link them together in the past. From the\npoint of view of the linker, they had different names:\n\n(1) Quick RT: art_set32_static_from_code\n(2) LLVM RT: _ZN3art27art_set32_static_from_codeEjPNS_14AbstractMethodEi\n\nPreviously for portable, ExtractCodeAndPrelink would have translated\nthe art_module.ll names referenced by portable code at compile time to\npoint to a runtime table populated with the mangled names.\n\nHowever, when using MCLinker linker to link the object images\ngenerated by the portable compiler, the linker will only see the\nunmangled names. So now the have to be unique between quick and\nportable, so we\u0027ve adopted the naming convention of:\n\n(1) Quick RT: art_quick_set32_static_from_code\n(2) LLVM RT: art_portable_set32_static_from_code\n\nChange-Id: Id5bb9f8f1574433ada6b1d6734f509cca26b98a5\n" }, { "commit": "693ff61274cd2c9b8eb7e68c370f84a911b8ca52", "tree": "11f1f6b3e5302b8a58c5ca44c3a1bec1b16d90a4", "parents": [ "12a2bdd245ea866b665d4800b493397ec412892a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 01 10:56:12 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 01 21:29:49 2013 -0800" }, "message": "Performance improvements by removing a DMB and inlining.\n\nCorrect the version of CAS used by Mutex::Lock to be acquire and not release.\nDon\u0027t do a memory barrier in thread transitions when there is already a\nbarrier associated with the mutator lock.\nForce inlining of the hot thread and shared lock code, heavily used by down\ncalls and JNI.\nForce inlining of mirror routines that are used by runtime support and hot.\n\nPerformance was measured and improved using perf and maps.\n\nChange-Id: I012580e337143236d8b6d06c1e270183ae51083c\n" }, { "commit": "2dd0e2cea360bc9206eb88ecc40d259e796c239d", "tree": "9d619dc9508cbe73e4793bf6f08cbc761abfb48a", "parents": [ "ac21b797b3a425975d656d6b84a7b24401d35f42" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jan 24 12:42:14 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 30 16:48:31 2013 -0800" }, "message": "Directory restructuring of object.h\n\nBreak object.h into constituent files.\nReduce number of #includes in other GC header files.\nIntroduce -inl.h files to avoid mirror files #include-ing each other.\nCheck invariants of verifier RegTypes for all constructors.\n\nChange-Id: Iecf1171c02910ac152d52947330ef456df4043bc\n" }, { "commit": "af6e67a4816d2593586115b89faa659225363246", "tree": "e3b330d6ed994f82972da4e5b8c3d06be2c66eae", "parents": [ "a6f3aaf8ca5385d870c35d1555b7787d56a70c95" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 08:38:37 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 15:37:56 2013 -0800" }, "message": "Further proxy refactorings.\n\nFactor the crawling of the quick stack arguments into a common visitor.\nFactor the proxy invocation dispatch into common runtime support code,\nfix numerous bugs relating to GC in the LLVM runtime support with this.\nClean up BoxPrimitive to not use an in argument as an out.\n\nChange-Id: I7b12c8d88d5083614e480b8fb1d2f2ef7c0a51b7\n" }, { "commit": "262e251e56d66bd309c6165964ac0a0be24447fe", "tree": "89453b090d7213e21d6c79ad731511bd8d1094af", "parents": [ "4f3d94b4e3b40fa7aa2bb5007fa4bb5703189b61" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue Dec 11 09:46:43 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue Dec 11 10:05:24 2012 -0800" }, "message": "Fixed slow path for invoke-direct with null this pointer.\n\nMoved the null pointer check until after the class linker tries to\nresolve the method on the slow path.\n\nChange-Id: Ie0f9e279a98a7e3a40899fee92337ef454c9a468\n" }, { "commit": "d752132c73072084a3def9257cca4fcee76047b6", "tree": "92c7bc15db2328db7c14c31f77504a7e879e8467", "parents": [ "278b0db1ee55106486a7aecf39a581aac09fb619" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Nov 21 15:38:24 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Nov 21 15:38:24 2012 -0800" }, "message": "Various fixes to the interpreter. First 23 run-test tests pass.\n\n- Factored out code to throw stack overflow error into its own method.\n- Increased kStackOverflowReservedBytes to 10kB to support interpreter.\n- Reordered return type checks to prevent type resolution with an\n exception pending.\n- Fixed field checks so they pass if the field is static or the object\n is the declaring class.\n- Suppressed using the interpreter for proxy methods.\n\nChange-Id: Ide73ec928ab0aa7b31229c4e69679a35dd948e43\n" }, { "commit": "373c52fd70336531010268a2de4f7d3b0f236924", "tree": "782478126bd283faeaeef38e3d3be4e37bde5a2b", "parents": [ "17ffcab3a9428ccdc5ae6b2b42c001fe3df1f1d6" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue Nov 20 16:11:52 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue Nov 20 16:17:38 2012 -0800" }, "message": "Added CheckSuspend and UpdateDebugger to interpreter loop.\n\nMoved CheckSuspend so that the code doesn\u0027t need to be repeated in LLVM\nas well.\n\nChange-Id: I1073f82352593bf0d5f99b28d382e4687f3a0d90\n" }, { "commit": "155dfe958412b71ab2adcc21611217c22277ec69", "tree": "91db1d5a368c8b9bc26a432064a272ce2a788051", "parents": [ "bac4334bb222e24fb3a080b69de0262a39e397c2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 09 14:24:49 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 11 18:18:21 2012 -0700" }, "message": "More Gc statistics and logging\n\nAdded a few metrics to Gc performance measurements:\nMean GC throughput, total time spent in Gc, total time spent in allocation code, mean time per allocation.\n\nSlightly optimized object allocation.\n\nChange-Id: Ie68153a7100affafc4a01dab34b341077e754867\n" }, { "commit": "50b35e2fd1a68cd1240e4a9d9f363e11764957d1", "tree": "4f3c58b7b293380f19e4f33357cb352e3330772e", "parents": [ "8e9a1497f0d7da4d55f6e7ed8a7d96ba6db7222d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 04 10:09:15 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 05 11:07:39 2012 -0700" }, "message": "Explicitly pass Thread::Current to MutexLock and Alloc.\n\nChange-Id: I8b75bc0617915465f102815b32306aa7760dcae4\n" }, { "commit": "81d425b0b232962441616f8b14f73620bffef5e5", "tree": "cd1e46656269acf20e78817d675e56d9b9133e6c", "parents": [ "bfaf917edbb1de8d158c3615e0da8ac3143d10c8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 16:03:43 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 27 17:19:25 2012 -0700" }, "message": "Pass self to lock methods.\n\nThis avoids frequent recomputation of\nThread::Current/pthread_getspecific.\n\nAlso add a futex based reader/writer mutex that is disabled.\n\nChange-Id: I118fdb99ef1d1c4bfda6446ba3a0d8b6ab31eaee\n" }, { "commit": "66f19258f9728d4ffe026074d8fd429d639802fa", "tree": "fd94009774c6cbbb1528ea096e606133bd35f104", "parents": [ "a5e1e3d153990845d80cb8d013157210f11a473c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 18 08:57:04 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 21 17:32:56 2012 -0700" }, "message": "Change dex cache to be java object instead of array, add pointer to dex file in dex cache.\n\nGeneric clean up to facilitate having GDB macros for Pretty* helper functions.\n\nImproved cleanliness of DexCache since having it as an object array was not the best solution.\n\nFixed a bug in InOrderWalk caused by ResolveType sometimes allocating classes.\n\nRename C++ Method to AbstractMethod and add two new classes Constructor, Method which both inherit from AbstractMethod.\n\nRename done to have the C++ code be closer to the java code.\n\nChange-Id: I4995b4c5e47a3822192b08afa24a639d3b1f4da9\n" }, { "commit": "b726dcb581bf72da46527378ccb6889020f0e6e9", "tree": "c0383ed788652926e93147e50d659ed226519548", "parents": [ "fd678beb171a4686a4f2d53ca4188a4ade8fa54e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 08:57:23 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 09:06:19 2012 -0700" }, "message": "Rename GlobalSynchronization to Locks\n\nAlso address some review comments in common_throws relating to\nToStr\u003cInvokeType\u003e and exception detail messages.\n\nChange-Id: Ibf2c0f147689fa236d349bd7f01eed3c2522552b\n" }, { "commit": "87e552db94588455c081efd87dbde0cd96d02942", "tree": "02fc47be772d65e3ba4f2453f5d330b7dd111d1a", "parents": [ "c747cffa950b8398cb8844d47e2253214c855750" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 31 15:54:48 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Sep 01 00:23:33 2012 -0700" }, "message": "Make ICCE logic common, refactor throws.\n\nThere were 2 sets of ICCE logic in ClassLinker::ResolveMethod and\nMethod::CheckIncompatibleClassChange, merged into just 1 piece of logic\nin Method::CheckIncompatibleClassChange.\n\nMove Throw... routines into own file and make adding the dex location to\nthe detail message more thorough.\n\nChange-Id: I953dbfa3fed3767f35799b2f82b16637c437dbbe\n" }, { "commit": "9a4f003f9a6da8e4c6ee279bd2a13aac37648183", "tree": "ce78afe7c842c8ff99d52c0a09bfe758d67d037f", "parents": [ "cfb21d92796087963c092b69110a2e7ff7a03a55" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Thu Aug 30 16:17:40 2012 -0700" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Fri Aug 31 10:49:06 2012 -0700" }, "message": "Remove the THROW_VERIFICATION_ERROR op and supporting code.\n\nChange-Id: Idc40a2c379048c7e3d74f50b2bd765a507a417ce\n" }, { "commit": "2fc1427ee9c534ed44d72184ad6d74ea65f3d5b3", "tree": "7759b3b2f7e25ac66a2d05a48ac9ff918b95eb80", "parents": [ "b9a287cc1c1f4f303acc186b7c97756f326d3537" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Aug 30 10:56:57 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Aug 30 10:56:57 2012 -0700" }, "message": "Fix location of ICCEs and NoSuchMethodErrors.\n\nAlso add detail as to why verifier believes an ICCE.\n\nChange-Id: Ie16ea2847fb94e157969c560a780c9345b1e7441\n" }, { "commit": "08f753d5859936f8d3524e9e4faa6cee353873ea", "tree": "5679a9ad8be6827e2a10de282cd28e3d000917de", "parents": [ "b0b2be7688109cff548d2ddc8f02eecbad49ecd2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 24 14:35:25 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 28 11:34:56 2012 -0700" }, "message": "Avoid compile time rewriting of dex code by verifier.\n\nCompile time rewriting of dex code leads to dead code that is causing\nissues with the LLVM compiler. Make instantiation and incompatible class\nchange errors be detected in slow path field and invoke logic so its\nsafe for the compile time verification just to softly fail the effected\nclasses.\n\nThis change places incompatible class change logic into the class\nlinkers ResolveMethod and consequently changes a number of APIs.\n\nChange-Id: Ifb25f09accea348d15180f6ff041e38dfe0d536e\n" }, { "commit": "3d71d80626fdc0fe1f61c84bb52e9f7d96c95060", "tree": "b65d908c11f270e6f54e241f332b01aa27134d68", "parents": [ "cd05a6274782c50a2b990c18b5f8d5ae5e44d509" ], "author": { "name": "TDYa127", "email": "tdy@google.com", "time": "Wed Aug 15 03:47:03 2012 -0700" }, "committer": { "name": "TDYa127", "email": "tdy@google.com", "time": "Wed Aug 15 04:01:19 2012 -0700" }, "message": "Fix JNI compiler for portable path.\n\nChange-Id: Ic2cbfcf126137b1b72f3b51e7af5a9be87987737\n" }, { "commit": "00f7d0eaa6bd93d33bf0c1429bf4ba0b3f28abac", "tree": "6a2172ece15c4699e6c2a67ce76f019db0a9a21d", "parents": [ "634ea28f78c4a138e6a1de54eae8696095422415" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 19 15:28:27 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 14 10:45:52 2012 -0700" }, "message": "Global lock levels.\n\nIntroduce the notion of the mutators/GC being a shared-exclusive (aka\nreader-writer) lock. Introduce globally ordered locks, analysable by\nannotalysis, statically at compile time. Add locking attributes to\nmethods.\n\nMore subtly, remove the heap_lock_ and split between various locks that\nare held for smaller periods (where work doesn\u0027t get blocked). Remove\nbuggy Dalvik style thread transitions. Make GC use CMS in all cases when\nconcurrent is enabled. Fix bug where suspend counts rather than debug\nsuspend counts were sent to JDWP. Move the PathClassLoader to\nWellKnownClasses. In debugger refactor calls to send request and\npossibly suspend. Break apart different VmWait thread states. Move\nidentity hash code to a shared method.\n\nChange-Id: Icdbfc3ce3fcccd14341860ac7305d8e97b51f5c6\n" }, { "commit": "a4f94740f9d66b21f4bcd3a225c434aa29cfa323", "tree": "88e3834418ac3e12d2c270fed3a370bda2643d0b", "parents": [ "214d307f3d5bbfe6254ae9e96d504d3e665baa2c" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue May 29 16:28:38 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue May 29 16:28:38 2012 -0700" }, "message": "Finish cleanup of class/field/method caching.\n\nChange-Id: I289ae724cbd98487429275837d23b7b2d7096156\n" }, { "commit": "41005ddb5576b8630a1084fbb3979ffa602c0599", "tree": "af8d74573cc3a98c959566233d12bdb15251910c", "parents": [ "c3f8fa57268740f95a3c25e605bcab638b18d7b9" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed May 09 17:58:52 2012 -0700" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Tue May 15 15:44:18 2012 -0700" }, "message": "Fix x86 type conversions. test-art-host-oat passes.\n\n- test-art-host-oat enabled in test builds.\n\n- Created inline versions of float-to-int and double-to-int.\n\n- Fixed calls to helper functions for long-to-float/double and\n float/double-to-long.\n\n- Removed storePair and replaced its use with storeBaseDispWide (fixes\n iput-wide and aput-wide with doubles).\n\n- Renamed helper functions to have art_ prefix.\n\n- Fixed move-exception to store the result back.\n\n- Fixed floating point comparison when the destination reg is the same as\n a source. The typing would get confused since the source is floating\n point, but the result is int, and a clobber is needed to overwrite the\n expected result type. A similar thing happens in float/double-to-int.\n\nChange-Id: I0a876072254411aa42d6acadb8723be030727219\n" }, { "commit": "776ac1fa61237db645adb4370a4aab888530caf4", "tree": "fbdfe1dec5f1c890d479f700f2b11549d5574119", "parents": [ "42f302c1da4cf1abf935d7aee29ece8d90441e0c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Apr 13 23:36:36 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Apr 14 13:38:06 2012 -0700" }, "message": "Separate out more of LLVM build. Move verifier.\n\nMove the verifier to its own directory and break out major components.\nAvoid compiling unused files as part of the LLVM build, increase the use\nof unimplemented for LLVM.\n\nChange-Id: Icd3b1708363fb70ae99417d06ef8669d60446533\n" }, { "commit": "5bb8601175bbb9cd761c715f4ba04f84d65e913b", "tree": "441c62e467b6c7a355d9ad65ea88a923abbb409b", "parents": [ "933abf8ce64e522b1c45b191b796bf2208a760d9" ], "author": { "name": "TDYa127", "email": "tdy@google.com", "time": "Wed Apr 11 05:57:28 2012 -0700" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Wed Apr 11 11:01:21 2012 -0700" }, "message": "Fix 044-proxy. Implement proxy for now, working on x86 and ARM.\n\nAlready added a TODO to do the assembly code for x86 and ARM for proxy.\nUse LLVM .ll for multi-architecture now.\n\nChange-Id: Ibdeeee113dcf284592e9d7769d3044438cb1e453\n" }, { "commit": "9e5f5c1dc6cc6551db082fdcf484a145bfb684fd", "tree": "532222b9ea20b7bfd689adbb022d9206f13d556e", "parents": [ "a0f746b20d7587281df825689e36ab52af3d3149" ], "author": { "name": "Logan Chien", "email": "loganchien@google.com", "time": "Tue Apr 10 13:51:45 2012 +0800" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Tue Apr 10 09:14:39 2012 -0700" }, "message": "Implement throw-verification-error opcode in LLVM-based compiler.\n\nChange-Id: I3b902e593f380d7524c8a2d94800d0323364c613\n" }, { "commit": "3f9137d85b3c2c0800fb26e3415ace1a6b7cc4de", "tree": "7b2c72c1ccf3512fb52fc48ecfb84f9c5c680a82", "parents": [ "6ac6a365a9bdb00e85b46eb8fadfb0cebf653cba" ], "author": { "name": "TDYa127", "email": "tdy@google.com", "time": "Sun Apr 08 15:59:19 2012 -0700" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Tue Apr 10 04:49:44 2012 -0700" }, "message": "Fix NPE message in LLVM.\n\nChange-Id: Ie65060d065d747a6c9ad22c61d5fb29f6cf3c249\n" }, { "commit": "0045a290e6b79a274250e3112880c04bde437d4a", "tree": "a125d556b6c817218b14f6f3f72079b83d0032c4", "parents": [ "d23f5206cf72ca3fb53ac3049649bb7d018232bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 21:08:41 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 22:38:22 2012 -0700" }, "message": "Avoid a case of eager initialization.\n\nDon\u0027t mark as initialized classes that contain static field\ninitialization.\n\nChange-Id: Iedcabbdf355e8861eb7731650eee1467f68ae0cd\n" }, { "commit": "0f3c55331439970e01af67f80ac117c473bc04cf", "tree": "cfa28ad2a58af1ffddb0a2ce90443ad4606743c3", "parents": [ "273cf36d199cf73de3cf61a559ad27c9d23f9825" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 30 14:51:51 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 30 14:51:51 2012 -0700" }, "message": "Kill constants.h and fix some copyright headers.\n\nChange-Id: I51c04d731d6de035328781d8ac134ad6fcf49897\n" }, { "commit": "57b86d47b66322693a070185fadfb43cb9c12eab", "tree": "81d53304a63a660a683917a19f7dd83723a397ad", "parents": [ "68aefb06c449df0bd8a34f3491f7594ee3f9dd0b" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Mar 27 16:05:41 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 29 11:51:44 2012 -0700" }, "message": "Refactor runtime support.\n\nChange-Id: Id7470a4105838150d5ceb73ab2c8c83e739660df\n" }, { "commit": "fc9e6fabed89d948fa8c0e9d673e430076712c60", "tree": "12e01ab3a937729d9482ab7fded1f1e0ab73dbd7", "parents": [ "67d920071fe4a0aa8b8bc339e93b18276238c320" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Fri Mar 23 15:14:29 2012 -0700" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Mon Mar 26 16:06:01 2012 -0700" }, "message": "Compiler intrinsics\n\nAdd intrinsic support. Some of these appear to be of\nlimited value, so we may end up removing a few. In general,\nthe instrinsics provide small, but measurable, gains.\n\nOnly Arm is currently supported, but most of these should\nwork for our other targets as well.\n\nThis is an interim solution. My plan is to the intrinsic\nrecognition action up into the basic block building phase once\nwe start doing inlining.\n\nChange-Id: Ia2913f2cdecaa4e80469caf69dbf8e2f61d4506a\n" }, { "commit": "c0f0933249cf516b37717faa766e1e9808f7c1f8", "tree": "04aedbf3e8502637673b82cf181dfab1b5aca1a6", "parents": [ "942df41c03ddb9f7b47a34fcfc4414d8b8ca8ce1" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Mar 26 13:27:06 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Mar 26 13:27:06 2012 -0700" }, "message": "Add the missing link between compiled code and the debugger.\n\nWhen a debugger connects and disconnects, we now let compiled code know that we\nneed to be kept informed about what\u0027s going on.\n\nAlso fix a threading bug when threads exit with a debugger attached.\n\nAlso some minor tidying, mostly involving naming.\n\nChange-Id: Iba0e8b9d192ac76ba1cd29a8b1e6d94f6f20dea8\n" }, { "commit": "c6f3bb87ffbb44d902c4a1f67a71bb108bd01560", "tree": "c423db1d6e7914e57559ab5a4325094e6f1ea5c0", "parents": [ "e5b7894351ecc5ef99442eea0700c913178c95ce" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 21 20:40:33 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 21 20:41:16 2012 -0700" }, "message": "Further x86 progress and image creation.\n\nChange-Id: Idafadfc55228541536f25d2c92d40d9e0510b602\n" }, { "commit": "0ece7b950dc5821e79c930acf31fb7e73c570849", "tree": "ce4006468c5ea8b068055b5f770b058477e35a25", "parents": [ "e663a5b5b218deae32de739d89f888ceeeba65df" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 09 18:14:40 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sat Mar 10 09:51:21 2012 -0800" }, "message": "Remove an unnecessary \"art\" prefix on functions in the art namespace.\n\nChange-Id: I160bc78ef9000364f657a9972dd8c648ca2f53aa\n" }, { "commit": "a433b2e8edf91c8124abd8eb5b3bd558a40fc987", "tree": "d0c6b2cd101ed66897193ac03f4738d6a7473da3", "parents": [ "f5a9d80f6afaf714c1afeaf2e869ff194aa2556a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 09 08:40:33 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 09 08:59:48 2012 -0800" }, "message": "Tidy MIPS floating point helpers.\n\nFix initialization bug in thread.cc.\n\nChange-Id: Ibdc81d6f5c8b1a245044bf901003179167a5d4f1\n" }, { "commit": "adb8c67f6d87a160d4e3a8afea7cb93f6c14568b", "tree": "93b265743b7fd7a831803f6bc84e2aa8ce338156", "parents": [ "731b2abfccd8704d129e3b8e46a086660161fef3" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Mar 06 16:49:32 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Mar 06 17:21:07 2012 -0800" }, "message": "Move the compiler away from libdex.\n\nChange-Id: I0c5fdc38d94f1199023cfc6aab3060b927ad2383\n" }, { "commit": "11f9d2130e938511efceb6d2a4793cee7dfdde35", "tree": "cf6a63ad64a66c5ee225c789adc15ba4962d526c", "parents": [ "e8267506ac2d64351e21ce90e8643b9247e1803c" ], "author": { "name": "Bill Buzbee", "email": "buzbee@google.com", "time": "Sat Mar 03 20:03:18 2012 -0800" }, "committer": { "name": "Bill Buzbee", "email": "buzbee@google.com", "time": "Sat Mar 03 20:03:18 2012 -0800" }, "message": "Mips portable floating point support\n\nFleshing out the portable floating point.\n\nChange-Id: Ie7a6dcb168a4eb2a61a52e6d747da4118d54f3aa\n" }, { "commit": "1984651929744dd603fd082e23eacd877b9bc177", "tree": "ffa10973a88f0f2743f3fbfaf5e7e4235d5f262d", "parents": [ "f3c01c530ab3a645bd5a980522df4d396dcbf228" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 24 11:42:47 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 27 16:08:56 2012 -0800" }, "message": "Invoke static/direct dispatch change.\n\nThere was a subtle race in static/direct dispatch via the code and methods\ntable. This patch removes the table in preparation of architectures like\nx86 which will more aggressively sink loads.\n\nRemoves unused fields, code.. Brings back resolved methods table\nshort-cut and associated fast paths to use this (such as in Proxy). Adds\na resolution method that is used as the trampoline for static and direct\nmethods.\n\nAdd source file and line number to Throwable::Dump.\n\nMethodHelper knowledge of runtime methods.\n\nChange-Id: Ieae1b74c24072e6327a5bb2cad466f04e3c46c4d\n" }, { "commit": "8cd6ddaeb78cc904e13bac88753654a04b2e15b8", "tree": "005ebbe14563bb3ec5bfc9837bf3fd4284427cec", "parents": [ "a656a0f6fbcf2ba3f15cae54a773b9c636dd32c1" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Feb 22 10:15:34 2012 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Thu Feb 23 15:55:20 2012 -0800" }, "message": "Numerous fixes to compiler and verifier for cts vm-tests.\n\nClassNotFoundExceptions in ResolveType are converted to\nNoClassDefFoundErrors.\n\nCompiler checks for puts into final fields.\n\nMethod resolution searches direct methods if an appropriate virtual\nmethod can\u0027t be found.\n\nInvocations of \u003cclinit\u003e are rejected by the verifier.\n\nInvoke-super and invoke-virtual can\u0027t be used on private methods.\n\nUsing invoke-interface on non-interface methods and not using\ninvoke-interface on interface methods leads do an error.\n\nChange-Id: Ia589f1ffccf91b62812ee34c8c5fae1aaf3798c6\n" }, { "commit": "60db5ab3a2e480db9236325a14cb5a867881d8bb", "tree": "c4a514941c2d41b82581d2c3d83ecb742e575231", "parents": [ "6a91799974e992e915facb9d9372dc8847a09a64" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 20 17:02:00 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 20 18:15:35 2012 -0800" }, "message": "SIRT work around JNI support for legacy apps\n\nLegacy apps may be broken and expect a jobject to equal an Object*. This\nis a work around to support outgoing SIRT arguments being converted to\nbe Object*s. It steals the unused GC map pointer on native methods.\n\nChange-Id: I8fb3701b0cdf15be9e8f9b722e47386505482666\n" }, { "commit": "c8b306f5221658c7e4b5516be8917dc8c9288e7e", "tree": "d8d3499030152361129f002e99d2b4a10d36660d", "parents": [ "6c6f12d0ed64e0f4bcbb7139eca8fe564302f385" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 17 21:34:44 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 17 23:52:39 2012 -0800" }, "message": "Run time illegal access checks on static and direct methods\n\nFixes test 075.\n\nChange-Id: I28b20451dcae8000dc0e2cb9068dfa5166659d43\n" }, { "commit": "91bf6cd47174f5c17265320f7a350722720390a5", "tree": "f4823c09793162e00b2fdd43990e12f2837d7562", "parents": [ "f05d0dd011d3dbcf78bf35d0771f67371f5f6b58" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Feb 14 17:27:48 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Feb 14 17:27:48 2012 -0800" }, "message": "More debugger support.\n\nThis wires up method-entry events and fixes a bug in exception events.\n\nChange-Id: Ia7c46786a8073434fbb4546615072622f301ef84\n" }, { "commit": "f3773f0913f8b76eee2088bfd999aaca6a50e750", "tree": "d9c2f8ce0736a6260d96586cbb3fcdb75430d4b2", "parents": [ "a32a6fd4a781262dff4fec102da053d16b7ef6c0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 13 16:45:29 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 13 16:56:22 2012 -0800" }, "message": "Move finalizer registration into managed code\n\nAlso move Object.getClass into managed code.\n\nChange-Id: If32674ac496116aaa82c461d8e6196e27fa7b591\n" }, { "commit": "a32a6fd4a781262dff4fec102da053d16b7ef6c0", "tree": "5c2a35aec522902cfdf508a74898ee091c91f7ab", "parents": [ "d08547a802e6dbb71fde3ed3dac9aa0b03a5b17b" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 06 20:18:44 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Feb 13 15:41:26 2012 -0800" }, "message": "Runtime access checks on virtual method calls\n\nAt verification time we may not know that an illegal access or method\nnot found exception should be raised and so we defer the decision to\nruntime. When the decision is deferred we perform an appropriate slow\npath method invocation that can check for access violations.\n\nThis change also attempts to reduce code duplication, improve the\ndiagnostic information in exceptions, clean up field slow paths slightly\nand to move the slow path calls lower in the Thread class so that they\ndon\u0027t effect the offsets of data items when calls are added or removed.\n\nChange-Id: I8376b83dcd7e302cbbddf44c1a55a25687b9dcdb\n" }, { "commit": "44b412bb795fa6999129b2bc16f5eec1ea97e8f8", "tree": "bf2a45463f46c506130f319b1a176b49db4d5252", "parents": [ "959f8ed11ef9493a89b90a1f9a82a811dbc7ae0e" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Sat Feb 04 08:50:53 2012 -0800" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Tue Feb 07 18:19:52 2012 -0800" }, "message": "Codegen support for debugger\n\nAdd the ability to generate code with support for debugging.\nThis involves generating a callout to an equivalent of the\nold \"updateDebugger()\" before each Dalvik opcode, method\nentry and method exit.\n\nThe added code is fairly compact - 8 bytes per Dalvik opcode,\nplus 4 additional bytes per safe point.\n\nI dislike the idea of always making this call, so I\u0027m reusing\nthe dedicated register rSUSPEND to hold the address of the callout.\nrSUSPEND is normally used to reduce the frequency of full suspend\nchecks, but when debugging this isn\u0027t necessary - allowing\nus to resuse this register to hold the address of the callout.\nIf it is non-null we make the callout, otherwise we continue. I refresh\nthis register from a slot in the Thread structure on method entry\nand also following the return of taken suspend checks. In this way,\nthe debugger has the ability to control updates on a per-thread basis,\nand the performance penalty is greatly reduced for threads that don\u0027t\nhave any pending debugger requests. Once the debugger attaches,\nit would suspend all threads, walk through the thread list, set\nthread-\u003epUpdateDebuggerFromCode to art_update_debugger and then\nturn everything loose.\n\nOne thing I\u0027m not doing, though, is debugger updates before and after\ncalls to native methods. This is something that will have to be done\nby the stubs, because I don\u0027t know which invokes are native. Oh, and\nthere will also need to be an artUpdateDebugger call on the exception\npath.\n\nI\u0027m passing the DalvikPC to the stub, and am using special\ncodes (-1 and -2) to denote method entry and exit. The stub\nrecovers the current Method* and Thread* and passes them on to\nartUpdateDebugger().\n\nWhen we\u0027re compiling in this special mode, all optimizations which\nmight result in code motion or suppressed load/store of a Dalvik\nregister are turned off. No register promotion is done, so everything\nwill be in its home location.\n\nChange-Id: Iaf66f4d0d094a1699269d0a1ad1ed33e7613aef8\n" }, { "commit": "1bddec3a6521f16df37499754000a3b1787a52e9", "tree": "f605d0f642f2a62012d0f5a1bd0351afbaa8fa5a", "parents": [ "223f20fa939c386c695977263780dea2195093db" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Feb 04 12:27:34 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Feb 05 03:02:00 2012 -0800" }, "message": "Add access check slow paths to field accesses\n\nThis check makes the verifier, compiler and runtime agree with who\nshould perform access checks and fixes compliance for these\ninstructions.\n\nIntroduce new \"fast\" sget/sput that just get the static storage base\nfrom a method\u0027s declaring class when the static field is within the same\nclass. Saves a load and branch in the common case.\n\nFold gen routines for wide and not wide together.\n\nFix bug where sub-classes could appear intialized in the image but their\nparents were only verified.\n\nExtra debug output for test case 075.\n\nChange-Id: I934da3624ed8fa8e026b2c95d936d04b1af022ef\n" }, { "commit": "0eb7d7e2a0c2767b5fd86bd52436c618fd4c3120", "tree": "a446a35a4dfc5d258bb2817b92319b2f8ea84b20", "parents": [ "09cc2d39b3073ab0c613a9f660983e781693d958" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 31 21:12:32 2012 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 31 21:14:59 2012 -0800" }, "message": "Implement array allocation with access checks.\n\nChange-Id: I0b2c0274087f3f7ed5b0b71c441e5d7e5a25f819\n" }, { "commit": "2faa5f1271587cda765f26bcf2951065300a01ff", "tree": "c59557857c36741cfcd1b6bf5ae039886a8ad6e7", "parents": [ "0850c69c88d49efd4075288511b70d6c032bd0c8" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Jan 30 14:42:07 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Jan 30 16:46:29 2012 -0800" }, "message": "Fix copyright headers.\n\nChange-Id: I5b7bc76a370238d810d78522bd5531600746700f\n" }, { "commit": "f3778f66879fa038a7c9ebe46d5bc4167ddef7d8", "tree": "50499b6687d784cdcf571f5b6c5885d5f75be334", "parents": [ "aa56b7287426ffc476843ad2ffab9eb70709e4ab" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 26 14:14:35 2012 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Jan 26 15:14:55 2012 -0800" }, "message": "Fix test 046.\n\nLooks like a recent change put us on an untested code path, and we need\nto start distinguishing callers\u0027 intentions. We can also remove some\nunnecessary duplication.\n\nChange-Id: I950139e6fdc8656b89d8e1520e8b50f681c7f7b2\n" }, { "commit": "48d7222d2f543752ffb17db05e0b9e891061135c", "tree": "cf0295a04e1d32dce700620900bdc15a09d9913a", "parents": [ "34c77ad4fc67de52463544b78a35b86d15b496e4" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Jan 11 15:19:51 2012 -0800" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Jan 11 15:33:27 2012 -0800" }, "message": "Add unimplemented artResolveMethodFromCode\n\nChange-Id: I1ba804c56413477451abff82939e27579494e1f9\n" }, { "commit": "e343b76af81a005ef64f5e75a555389fd9147dab", "tree": "f6ec517c56338244d06723c49d610b5d948c3fa7", "parents": [ "5b245f344cbe8402cfe35c3a42988081e71208bc" ], "author": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Mon Dec 05 16:36:44 2011 -0800" }, "committer": { "name": "jeffhao", "email": "jeffhao@google.com", "time": "Wed Dec 07 16:34:28 2011 -0800" }, "message": "Stub to capture method entry/exit.\n\nAdded stubs to allow traceview to do method tracing. Currently only\noutputs to logcat, and a later change will generate the proper log file.\n\nChange-Id: Icaafc50e2eaf042ddc4d882011f7e8121bdd8b1c\n" }, { "commit": "b093c6b27f8ea9bbe2d49c03ebe345203a121199", "tree": "34e49f46eb30a2aee8590dbf11365c8b5824c16b", "parents": [ "95e4ef7cebf2e33b6e73a3957089f958e41771f5" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 31 16:19:55 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 31 16:19:55 2011 -0700" }, "message": "Support for access check in checkcast and instanceof.\n\nChange-Id: Ie15216618b35cace7d351be2b0a1c466ed6db489\n" }, { "commit": "28ad40dc3ec2f09b0ffd4f6d6787bf1b532ccd5d", "tree": "f5628e47948a0d255112a3a7a01b67db41c8d424", "parents": [ "e0918556e7551de638870dcad3f2023f94f85a50" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 27 15:19:26 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 28 23:14:15 2011 -0700" }, "message": "Support for unresolved types in new-instance during verification.\n\nAlso, ensure that classes that don\u0027t load are erroneous, warn early\nabout exceptions left on a thread by the verifier/compiler, factor out\nslowpath checks for the compiler and fix the slowpath selector for\nconst-class.\n\nThis change causes more dex cache misses at runtime (more slowpath\nexecution). It also requires a \"mm clean-oat\".\n\nChange-Id: I014b49ebdd7d8f7dd2e39cc0958fc0b708d58c4c\n" }, { "commit": "cc4540ece917ef4b5c81ee684e3294e47e49ff5b", "tree": "a43560b6c3f70708cad676476e7aca395c10e337", "parents": [ "baba887a4f5d2a454ab3f8f36a26346f8fdd0670" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Thu Oct 27 13:06:03 2011 -0700" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Thu Oct 27 13:10:05 2011 -0700" }, "message": "Add slow path for OP_NEW_INSTANCE\n\nIf the type is unresolved at verification time, we can\u0027t be sure\nit\u0027s a valid new. The compiler will now check for type resolution\nat compile time. If unresolved, we\u0027ll call out to the slow path,\nwhere the access check will take place.\n\nThis CL only contains the compiler portion of this change. For\nnow, the slow path code just calls out to the normal routine.\n\nChange-Id: Ia568a164389baedaa695a44f6845a0fdfe585b56\n" }, { "commit": "6a4be3a6226cec645cf905dd352e44f7968a7fa4", "tree": "15c98c404db44f60850a94d9b3cff29ff16e1cd5", "parents": [ "fb8a127fd2ca8eb6cbbc9d62b708c1fc2b3003a4" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Oct 20 16:34:03 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Oct 20 17:01:32 2011 -0700" }, "message": "Fix calling conventions for UnresolvedDirectMethodTrampolineFromCode\n\nChange-Id: I1b35c33d89f0526c80bc8d21e064617d0eac6cd1\n" }, { "commit": "6fd03fb67e4628689f0abf34edeacc0e35b6295e", "tree": "89e287870dfb98b31f2ea7336166801dad5cd55a", "parents": [ "29405cfc691196c21325e80d9a9f3795026eff4e" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Oct 17 16:11:00 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Oct 17 16:21:03 2011 -0700" }, "message": "Fix ObjectInitFromCode to do callee frame setup\n\nChange-Id: I70512c85a3a810fd672495b86655c215a07bb977\n" }, { "commit": "466bb25416b88fabd5d4387b7c7e5cc1ece78b8c", "tree": "8d0952cbd850f1d0ac2e43a8372ce571fb4982d1", "parents": [ "a10cd94bb77ed66fa0a8b66141c4504045c92d30" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 14 03:29:56 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 14 21:45:27 2011 -0700" }, "message": "Proxy implementation\n\nThis rounds out the proxy implementation by adding missing pieces to the\nclass linker, extending tests and fixing issues in the runtime support.\nThere are also some tweaks for performance and to clean up Method/Object\na little.\nA unit test of the functionality is \"art/test/run-test 044\"\n\nChange-Id: Id94102d10b81cd9b12b95ba8618f6187490204c4\n" }, { "commit": "caab8c4ef372db5c119bfac1911fa27b174a935c", "tree": "375a1e284b123b91d4960aa674b223e1eea0583a", "parents": [ "14b1b249bdd21d674c9371dee34f8ae055f73ba2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 12 12:11:18 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 12 18:12:21 2011 -0700" }, "message": "Various tidy ups, add LIKELY/UNLIKELY macros.\n\nMake more aggressive use of the dex cache when we have it.\n\nChange-Id: I125a7bac031f7f0cec68194b5380bdfd83c92833\n" }, { "commit": "6f495f2898a418f87e2a919e04fe23521bb0b9e9", "tree": "6ba9e7bedb3425095725ea1ec514c279839e8511", "parents": [ "98449a2674ea437eeae53327bf75918c8276a96b" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Oct 10 15:05:03 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Oct 10 15:05:03 2011 -0700" }, "message": "Clear results registers on long jump\n\nChange-Id: Ie4f84ad60b155bc27016ec15d4ecfcce8b0117f1\n" }, { "commit": "aded5f7ab991f3c1132851599d3bc60ff6707eed", "tree": "7fb5e92c224adf98cadd67b7df99cc2d2851cbcb", "parents": [ "bcbaaf3df56178263dbc110dcb0d1cf6c9d167c3" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Oct 07 17:15:04 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Oct 10 12:51:28 2011 -0700" }, "message": "Working ClassLoader\n\nChange-Id: Ia1122165e47f846a1d4506111849f830d9f14c1b\n" }, { "commit": "4a510d8750d50b77353d0a5a891d1b9b3a48ecd5", "tree": "f6df9490fee55d2eae88b2281993e94953c6a805", "parents": [ "4a14bc2e374add8f7df8232b3af3d8881eb4c84b" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Oct 09 14:30:24 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Oct 09 14:30:24 2011 -0700" }, "message": "Ensure R4 is initialized for upcall. Sort suspend naming.\n\nChange-Id: I73eb49e1af2ced971eeaafb7c07f5d964dae1824\n" }, { "commit": "ce9eca6de042f26e9eebc41c9bee8b4d14f753aa", "tree": "7a141da8787f6653b30595f478ec656fd8dd01d6", "parents": [ "6384f644e8120a15bddb9a72976eac1c158aba16" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 07 17:11:03 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 07 18:24:23 2011 -0700" }, "message": "Move from code field routines to runtime support.\n\nAdd support for exceptions from these routines. Use a macro for pending\nexception delivery. Fix bug in object_test where the signature of a\nFromCode function had changed.\n\nChange-Id: If486eff8659e5f6bb38d95b1139e8981656e4fb0\n" }, { "commit": "4f0d07c783afef89703dce32c94440fc8621a29b", "tree": "f6f4b68d5afc6551c61d3e86153878d9891ea13a", "parents": [ "3ddac99d4dc6a036fac59d8f0bdc664ef619fb04" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 06 23:38:47 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 07 11:40:56 2011 -0700" }, "message": "Spill possibly reference holding registers for runtime calls.\n\nLive references may be in registers when we crawl a stack for GC during\na runtime call. Whilst an exception won\u0027t occur we need to save the\ncallee save registers into the frame so the stack crawl can find their\nvalues. Create variants of the callee save method to save just the\nregisters that are necessary.\n\nChange-Id: I6fa479bffcbc333fe846f9bd3ef2e626e0209ed9\n" }, { "commit": "1cb0a1dfc32531c79a968aeac26ccb5525862497", "tree": "61d0345bd2b9b951026c9ce18d51c2b2d6963255", "parents": [ "40c47b50f6c37ff2af919a0d51f12d3e7cc794a3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 06 15:24:35 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 06 17:27:15 2011 -0700" }, "message": "Support for resolving unknown direct/static methods.\n\nIf we can\u0027t resolve a method we don\u0027t know whether it is direct or\nstatic from the dex information (other than the invocation instruction).\nAdd support for a third type of resolution stub that can discover the\ntype of the method based on the calling method and PC of the invocation\ninstruction. Its still unimplemented to look up the instruction and\nfigure out if the type is static or not.\n\nChange-Id: I8b76e6ba2c946376e7fe287dbcca17bcaab0e133\n" }, { "commit": "ad25ac568407ceb14334e8551dd1c4dd0fd6993c", "tree": "f1e2edb4b87d0b5ce35f94069db2f6cdb77db210", "parents": [ "8c2f6414e1cff9a024a14369352f7ce112528b9e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 04 19:13:33 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 05 22:40:00 2011 -0700" }, "message": "Static and direct resolution stub.\n\nEnsure that invoke static and direct go through a stub that causes\nresolution and initialization.\n\nChange-Id: I872900560322817d8f4378b04ac410d9ea0b3b17\n" }, { "commit": "ce30293d222c864fa281da98bc896dd1c98a9a16", "tree": "b56c3a7f8be3349aa50c78675e6959e51344938d", "parents": [ "20cde9033d51103f31e21436e88f80e1170c78ad" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Tue Oct 04 14:32:18 2011 -0700" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Oct 05 09:22:57 2011 -0700" }, "message": "Compiler cleanup\n\n o Added slow path for string resolution\n o Removed dead throw internal and runtime error\n o Restructured debug and optimization disable flags for make it easier\n for command-line option support.\n o Removed/converted #if 1/0 blocks\n\nChange-Id: I65fc561a55437b3f74d0dfff5af87f938008d70e\n" }, { "commit": "b408de744566a5c5a80be1ba7f5c88407e816945", "tree": "fa282edb7134f16de6e3ff81181e6d9724039ab3", "parents": [ "4377a7e93b47386de1e0c2e472a6e3e99d235331" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Oct 04 14:35:05 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Oct 04 14:51:27 2011 -0700" }, "message": "Implement the \"unreasonable array allocation\" OutOfMemoryError.\n\nThis doesn\u0027t fix test 061 because we still need AllocWithGrowth, but at least\nit gets us far enough to need that.\n\nChange-Id: Ia7b4a1f91a31e25d439f36b17280ce21c9ed8933\n" }, { "commit": "2d831014d88e38c0c499ce8597dcdb17b9d4c4b9", "tree": "793b222f8005580af855692045f7fe2804dca8d4", "parents": [ "5d1ac920fdaef5d4ec8f66bb734488cd9660b024" ], "author": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Wed Sep 28 22:06:53 2011 -0700" }, "committer": { "name": "Shih-wei Liao", "email": "sliao@google.com", "time": "Thu Sep 29 18:39:13 2011 -0700" }, "message": "Code refactoring: Split thread.cc into thread.cc and runtime_support.cc.\n\nChange-Id: I8484ee2993a2eb927ebba1f7b9db4709d16dad00\n" } ], "next": "c0c8dc8e677957d167066a2d06973126f108a6aa" }