)]}' { "log": [ { "commit": "cf7f19135f0e273f7b0136315633c2abfc715343", "tree": "ffa4d9efd9c45f4b6789acc1f534bb9327052b7e", "parents": [ "aea6888b056be21adf762e066c7f33b8939b8a06" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 22:06:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 22:21:57 2014 -0700" }, "message": "C++11 related clean-up of DISALLOW_..\n\nMove DISALLOW_COPY_AND_ASSIGN to delete functions. By no having declarations\nwith no definitions this prompts better warning messages so deal with these\nby correcting the code.\nAdd a DISALLOW_ALLOCATION and use for ValueObject and mirror::Object.\nMake X86 assembly operand types ValueObjects to fix compilation errors.\nTidy the use of iostream and ostream.\nAvoid making cutils a dependency via mutex-inl.h for tests that link against\nlibart. Push tracing dependencies into appropriate files and mutex.cc.\nx86 32-bit host symbols size is increased for libarttest, avoid copying this\nin run-test 115 by using symlinks and remove this test\u0027s higher than normal\nulimit.\nFix the RunningOnValgrind test in RosAllocSpace to not use GetHeap as it\nreturns NULL when the heap is under construction by Runtime.\n\nChange-Id: Ia246f7ac0c11f73072b30d70566a196e9b78472b\n" }, { "commit": "aea6888b056be21adf762e066c7f33b8939b8a06", "tree": "bfa17655ed3060b41e1cfd1583e590e9d9042f0a", "parents": [ "b08f4dcf90215ed49e0b796ab3e609bd605be8ba", "c7dd295a4e0cc1d15c0c96088e55a85389bade74" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 19:10:23 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 22 19:10:24 2014 +0000" }, "message": "Merge \"Tidy up logging.\"" }, { "commit": "c7dd295a4e0cc1d15c0c96088e55a85389bade74", "tree": "0c08a2236bc9ba5d9a4dc75d4dd0ed2d76f8f1c6", "parents": [ "94e5af8602150efa95bde35cc9be9891ddf30135" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 21 23:31:19 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 12:06:23 2014 -0700" }, "message": "Tidy up logging.\n\nMove gVerboseMethods to CompilerOptions. Now \"--verbose-methods\u003d\" option to\ndex2oat rather than runtime argument \"-verbose-methods:\".\nMove ToStr and Dumpable out of logging.h, move LogMessageData into logging.cc\nexcept for a forward declaration.\nRemove ConstDumpable as Dump methods are all const (and make this so if not\ncurrently true).\nMake LogSeverity an enum and improve compile time assertions and type checking.\nRemove log_severity.h that\u0027s only used in logging.h.\nWith system headers gone from logging.h, go add to .cc files missing system\nheader includes.\nAlso, make operator new in ValueObject private for compile time instantiation\nchecking.\n\nChange-Id: I3228f614500ccc9b14b49c72b9821c8b0db3d641\n" }, { "commit": "b556761d14e8dd0d41f1cc0f7d19726fe3497e8f", "tree": "5c1ad1cbe81e3fe4f23de7e34e3444bd034ca245", "parents": [ "70f4b9929048e71c4231d7976080be6277c3374b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 22 10:25:24 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 22 10:28:50 2014 +0100" }, "message": "Fix bug in UnionIfNotIn.\n\nBug: 18066207\n\nChange-Id: Ib9b24802546403b3d5a4da19996034eb45601f53\n" }, { "commit": "6f3dbbadf4ce66982eb3d400e0a74cb73eb034f3", "tree": "f7a20779e4d665f948c5fbcd26dac0071dafb8d4", "parents": [ "2df6840f68dd18d7dd8dbf53f8b6181bbfdc4fc4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 14 17:41:57 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 19:27:28 2014 -0700" }, "message": "Make ART compile with GCC -O0 again.\n\nTidy up InstructionSetFeatures so that it has a type hierarchy dependent on\narchitecture.\nAdd to instruction_set_test to warn when InstructionSetFeatures don\u0027t agree\nwith ones from system properties, AT_HWCAP and /proc/cpuinfo.\nClean-up class linker entry point logic to not return entry points but to\ntest whether the passed code is the particular entrypoint. This works around\nimage trampolines that replicate entrypoints.\nBug: 17993736\n\nChange-Id: I5f4b49e88c3b02a79f9bee04f83395146ed7be23\n" }, { "commit": "6e88ef6b604a7a945a466784580c42e6554c1289", "tree": "1e296564787b51514cf2eca5b732647c1a82912e", "parents": [ "58e51f38e2304a08aa9ec380383e0b3614f96a96" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 14 15:01:24 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 14 15:43:21 2014 -0700" }, "message": "Change MemMap::maps_ to not be global variable\n\nRuntime.exit() was causing globals to get destructed at the same time\nthat another thread was using it for allocating a new mem map.\n\nBug: 17962201\nChange-Id: I400cb7b8141d858f3c08a6fe59a02838c04c6962\n" }, { "commit": "7223d44a4893522e90d00bca38b119f710e55122", "tree": "c0bbe3f7b3cf04750b4b5359a03c2d63fc90978b", "parents": [ "3f5b8a1e7dfdf010591e17396d1d126740c5706c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 20:05:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 20:05:39 2014 -0700" }, "message": "Switch to C++11 style [[noreturn]].\n\nAlso remove old comment about calls to Abort getting merged. We have few\nLOG(FATAL)s so merging is less of an issue. Also, recompiling with O0 and\nsimilar will avoid the optimization and we expect that for GDB debugging.\n\nChange-Id: I840dc6f4b8511294be0e117c634ec15b745e7be5\n" }, { "commit": "647b1a86f518d8db0331b3d52a96392b7a62504b", "tree": "7370f795ef3c7fbdd2695d23bc6f8171f40f43f1", "parents": [ "acfbbd4df2fc1c79a7102587bebf398f95b5e5de" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 11:02:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 12:26:02 2014 -0700" }, "message": "Fix 2 new sets of clang compiler warnings.\n\nFix issues that are flagged by -Wfloat-equal and -Wmissing-noreturn.\nIn the case of -Wfloat-equal the current cases in regular code are deliberate,\nso the change is to silence the warning. For gtest code the appropriate fix is\nto switch from EXPECT_EQ to EXPECT_(FLOAT|DOUBLE)_EQ.\nThe -Wmissing-noreturn warning isn\u0027t enabled due to a missing noreturn in\ngtest. This issue has been reported to gtest.\n\nChange-Id: Id84c70c21c542716c9ee0c41492e8ff8788c4ef8\n" }, { "commit": "fc787ecd91127b2c8458afd94e5148e2ae51a1f5", "tree": "ef48c0f511ee9bf4ed85607cc4d530bace7e6cae", "parents": [ "8fa8c904f7c783204a1dc9438429391d256658da" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 21:56:44 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 22:22:46 2014 -0700" }, "message": "Enable -Wimplicit-fallthrough.\n\nFalling through switch cases on a clang build must now annotate the fallthrough\nwith the FALLTHROUGH_INTENDED macro.\nBug: 17731372\n\nChange-Id: I836451cd5f96b01d1ababdbf9eef677fe8fa8324\n" }, { "commit": "13735955f39b3b304c37d2b2840663c131262c18", "tree": "0a731ac42b8230f9929172fa3e3d8051874e2b18", "parents": [ "25b18bbdaa36ff936eb44f228f0518d4223e9d52" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 12:43:28 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 16:05:58 2014 -0700" }, "message": "stdint types all the way!\n\nChange-Id: I4e4ef3a2002fc59ebd9097087f150eaf3f2a7e08\n" }, { "commit": "9e878d50567f624094f3c4940ac3aedbc5eff3b9", "tree": "3b4f118d07fcdd8d64308e847e0140c0115448bf", "parents": [ "51205042343fa4852e01919de01fdcf261af510b", "0279ebb3efd653e6bb255470c99d26949c7bcd95" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 15:52:50 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 09 15:52:51 2014 +0000" }, "message": "Merge \"Tidy ELF builder.\"" }, { "commit": "0279ebb3efd653e6bb255470c99d26949c7bcd95", "tree": "d58b29754f7b3c88616e6e4d6c19346821d244ae", "parents": [ "f1f05d303988a5c071c87b760056be8358276c94" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 17:27:48 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 08:25:34 2014 -0700" }, "message": "Tidy ELF builder.\n\nDon\u0027t do \"if (ptr)\". Use const. Use DISALLOW_COPY_AND_ASSIGN. Avoid public\nmember variables.\nMove ValueObject to base and use in ELF builder.\nTidy VectorOutputStream to not use non-const reference arguments.\n\nChange-Id: I2c727c3fc61769c3726de7cfb68b2d6eb4477e53\n" }, { "commit": "5869a2c27ee0dbd7b420614c76ff14a706f0c5fb", "tree": "39bb17a7ae4a4d6c1db99d1a5e07ede899da0aec", "parents": [ "f1f05d303988a5c071c87b760056be8358276c94" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 08 14:26:23 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 08 14:26:23 2014 -0700" }, "message": "Dump mutator lock for thread suspend timeout\n\nShould help us see if thread suspend timeouts are due to someone\nholding on the mutator lock for too long or if it is a missed\nfutex wake.\n\n(cherry picked from commit f924d2381a5ea53967ba1e279766d601b9be05ea)\n\nBug: 17837911\nChange-Id: I3417156a78f574076d5da17c86836c5a7c76084a\n" }, { "commit": "7e70b002c4552347ed1af8c002a0e13f08864f20", "tree": "79d5ee5444a5be70130d9a75dd51831c4b15687b", "parents": [ "edc34c88b8f8abd04f9c4668787403608cf0b2d4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 11:47:24 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 11:57:59 2014 -0700" }, "message": "Header file clean up.\n\nRemove runtime.h from object.h.\nMove TypeStaticIf to its own header file to avoid bringing utils.h into\nallocator.h.\nMove Array::DataOffset into -inl.h as it now has a utils.h dependency.\nFix include issues arising from this.\n\nChange-Id: I4605b1aa4ff5f8dc15706a0132e15df03c7c8ba0\n" }, { "commit": "07140838a3ee44a6056cacdc78f2930e019107da", "tree": "4f344df6ec2612d8ddbf0a00e2b49a39a7501249", "parents": [ "f9098ea6d7326b4d8de2860d18adf74046b80855" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Sep 30 15:43:59 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Sep 30 15:43:59 2014 -0700" }, "message": "Enable -Wunreachable-code\n\nCaught bugs in DeoptimizeStackVisitor and assemble_x86 SIB encoding.\nAdd UNREACHABLE macro to document code expected to be unreachable.\nBug: 17731047\n\nChange-Id: I2e363fe5b38a1246354d98be18c902a6031c0b9e\n" }, { "commit": "9ef78b59da51080882e47505896b420977fd79ae", "tree": "6c568756e4e16e68d5d3346261009350969d5b77", "parents": [ "95f03e6a4737f90685fab86e98709f1c4393d5ef" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 25 17:03:12 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 16:04:56 2014 -0700" }, "message": "Fix broken runtime SetStatsEnabled logic\n\nPreviously, Runtime::SetStatsEnabled wouldn\u0027t take stats_enabled_\ninto account when deciding whether or not to increment / decrement\nteh stats enabled counter. This resulted in counter underflows and\nother errors which caused some CTS tests to fail.\n\nAlso added some locking to prevent race conditions.\n\nBug: 17360878\n\n(cherry picked from commit a98ffd745bbecb2e84a492194950c0b94966546b)\n\nChange-Id: I21d241a58d35bd6a607aa2305c6da81720bd0886\n" }, { "commit": "02875c56e80500fb0cc4671e33f57968232222ee", "tree": "588c48b5205a6697b810a116dc906eb615436c11", "parents": [ "6b604a1b0289e5e7211c2e5f8c4f395f51de7c3d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 17:36:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 17:36:39 2014 -0700" }, "message": "Fix some more pedantic compiler warnings.\n\nArgument name shadows field name.\nLack of case statements for kMips64.\n\nChange-Id: Ib3ca16f79a4d44ae80ef32f22cc79cbe9527f4f1\n" }, { "commit": "6b604a1b0289e5e7211c2e5f8c4f395f51de7c3d", "tree": "2789123796907a61bdfbc9c6f5727ac6cda8f559", "parents": [ "e7e34d08cffbd9a43ca9b097f8c7fdd54fbeaa27" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 15:35:37 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 16:51:29 2014 -0700" }, "message": "Fix Class::IsInSamePackage to not read beyond the end of a StringPiece.\n\nFix length of string piece to be size_type rather than int because expecting\nnegatively sized strings is a bad idea and we should use unsigned types to\ndefensively guard against mistakes.\nRemove max_size and capacity since the return type is inconsistent between\nGoogle and the STL, and we don\u0027t need the functions.\nAdd a bound check in libartd to operator[] accesses.\n\nChange-Id: I1b87a03d8fbd95e7dbb106745e304d1083898075\n" }, { "commit": "cbd49ac468a70808a5c66ed4f04c80bc486dafce", "tree": "b36538ecc09feb2b80ef233e593c4ba121d2954f", "parents": [ "e570adabc38c6a454de62a0f912fd618f7685601", "69dbec6d9d55eeb2867949c2791d01dc9aa916c8" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Sep 17 21:56:53 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 17 21:56:54 2014 +0000" }, "message": "Merge \"Avoid suspending for alloc trace enabling when already suspended.\"" }, { "commit": "69dbec6d9d55eeb2867949c2791d01dc9aa916c8", "tree": "be4eca8161eafb4a8466f5adf28a2c8f27cd37d1", "parents": [ "926d8e99aa69151a271180299df68963688010d8" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Sep 15 18:03:41 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Sep 17 14:31:55 2014 -0700" }, "message": "Avoid suspending for alloc trace enabling when already suspended.\n\nBug: 17499772\n\n(cherry picked from commit 1d6ee090fddd4bfd35c304d6ceb929d5c529dfcc)\n\nChange-Id: Id09809c476c685f0a197ee75bb08638931364efd\n" }, { "commit": "58553c7fd89ce69857017322444265469bb6af62", "tree": "fd479699cb7919c82e4c05fbc7a249b01262befe", "parents": [ "a20b7b3ecf90bb761d7085403782721f2fb474c5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 16 16:25:55 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 16 17:20:13 2014 -0700" }, "message": "Add allocation tracking allocators to ROSAlloc\n\nUsed to monitor native memory usage, results are approximately\n12-100KB memory per app.\n\nChange-Id: If5a46cd8d543851948a8cb69487f3044965b44ce\n" }, { "commit": "2cbaccb67e22c0b313a9785bfc65bcb4b25d0676", "tree": "daeb766e19880b651fd9c4a719c9a07dd7d4bd0e", "parents": [ "bace0378d720a1d2938ec7f6be17e2814671d20a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Sep 14 20:34:17 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Sep 16 12:50:08 2014 -0700" }, "message": "Avoid printing absolute addresses in oatdump\n\n- Added printing of OatClass offsets.\n- Added printing of OatMethod offsets.\n- Added bounds checks for code size size, code size, mapping table, gc map, vmap table.\n- Added sanity check of 100k for code size.\n- Added partial disassembly of questionable code.\n- Added --no-disassemble to disable disassembly.\n- Added --no-dump:vmap to disable vmap dumping.\n- Reordered OatMethod info to be in file order.\n\nBug: 15567083\n\n(cherry picked from commit 34fa79ece5b3a1940d412cd94dbdcc4225aae72f)\n\nChange-Id: I2c368f3b81af53b735149a866f3e491c9ac33fb8\n" }, { "commit": "a5a53efea976af505f4f849b5925d5e14c4f8e5c", "tree": "455a536a9394908c17ec1ede04ab7ec49f9d3420", "parents": [ "91bbd5347b391278b56440b2dbb0b77526bae4e8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 12 12:58:05 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Sep 14 14:48:24 2014 -0700" }, "message": "Add native support for FinalizerList.makeCircularListIfUnenqueued\n\nCalled from FinalizerReference.enqueueSentinelReference to prevent\na race where the GC updates pendingNext of the sentinel reference\nbefore enqueueSentinelReference.\n\nBug: 17462553\n\n(cherry picked from commit 3256166df40981f1f1997a5f00303712277c963f)\n\nChange-Id: I7ad2fd250c2715d1aeb919bd548ef9aab24f30a2\n" }, { "commit": "7b078e8c04f3e1451dbdd18543c8b9692b5b067e", "tree": "414229c6b87eb20ea24c40780752da5a3999a49a", "parents": [ "f79ba17defbd9342e44ab9f3de0807054673d3c9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 10 14:44:24 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 14:57:53 2014 -0700" }, "message": "Compile time performance improvements focusing on interpret-only.\n\nReduce virtual method dispatch in the method verifier and make more code\ninline-able.\nAdd a StringPiece with const char* equality operator to avoid redundant\nStringPieces and strlens.\nRemove back link from register line to verifier and pass as argument to reduce\nsize of RegisterLine.\nRemove instruction length from instruction flags and compute from the\ninstruction, again to reduce size.\nAdd suspend checks to resolve and verify to allow for more easy monitor\ninflation and reduce contention on Locks::thread_list_suspend_thread_lock_.\nChange ThrowEarlierClassFailure to throw pre-allocated exception.\nAvoid calls to Thread::Current() by passing self.\nTemplate specialize IsValidClassName.\nMake ANR reporting with SIGQUIT run using checkpoints rather than suspending\nall threads. This makes the stack/lock analysis less lock error prone.\nExtra Barrier assertions and condition variable time out is now returned as a\nboolean both from Barrier and ConditionVariable::Wait.\n\n2 threaded host x86-64 interpret-only numbers from 341 samples:\nBefore change: Avg 176.137ms 99% CI 3.468ms to 1060.770ms\nAfter change: Avg 139.163% 99% CI 3.027ms to 838.257ms\nReduction in average compile time after change is 20.9%.\nSlow-down without change is 26.5%.\n\nBug: 17471626 - Fix bug where RegTypeCache::JavaLangObject/String/Class/Throwable\ncould return unresolved type when class loading is disabled.\nBug: 17398101\n\nChange-Id: Id59ce3cc520701c6ecf612f7152498107bc40684\n" }, { "commit": "306db81aba41eb244a4e8299cf58ac18ae9999c7", "tree": "a069a9dda1b246466cbeee2736f46f18ef56714e", "parents": [ "b14339904c9cacc4af74260c7325e4eb32947f95" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 05 13:01:41 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 05 13:24:24 2014 -0700" }, "message": "Fix numerous issues with DdmVmInternal allocation tracking\n\nIssues addressed:\n- Using without JDWP attached caused native crash.\n- When buffer is full (64k entries), number of entries reported was 0.\n- Disabling tracking after disabling tracking caused native crash.\n- Asking for allocations after disabled caused native crash.\n- Lock ordering issues between mutator lock and alloc tracker lock.\n\nAdding 098-ddmc test to cover these cases.\n\nBug: 17392248\n\n(cherry picked from commit a5815065ac0877add9c0db3605d27b4d6c426e61)\n\nChange-Id: Ib0bc18dfcdafcc050ab9dceed3d167dd878d1d7a\n" }, { "commit": "ed2be1725fb79075892b1a9103487c9d9a95b350", "tree": "8adf1d95f401c2c7e7df1c659c0dfcd4dd27a68f", "parents": [ "645f9871115e82ff836397f6470cca825befd245" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 19 15:33:43 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 02 19:28:34 2014 +0200" }, "message": "Reduce lock contention when debugging\n\nUses a ReaderWriterMutex for the breakpoint lock to reduce contention during\ndebugging session.\n\nAlso adds missing thread safety annotations on fields and methods related to\ninstrumentation and debugging.\n\nBug: 16814665\nBug: 11667502\n\n(cherry picked from commit 59d9d668d4f4286813afe2b4e7c6db839222ce96)\n\nChange-Id: I5f1156da8c6cc8316d6db16e6cfb2470fe289ad3\n" }, { "commit": "bad0267eaab9d6a522d05469ff90501deefdb88b", "tree": "1ce689b75184cb322b9edde34d7af938c658a6ae", "parents": [ "0772a9d99c8774463c9076824340eaa4eb66f47f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 25 13:08:22 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 29 18:57:35 2014 -0700" }, "message": "Add native memory accounting through custom allocator.\n\nAdded a custom allocator that lets you pass in a special tag which\nspecifices where the allocation came from. This is used when\ndumping. The performance overhead is low since each allocation only\ndoes a atomic add/sub for each allocation/free.\n\nThe measurements are dumped to traces.txt during SIGQUIT.\n\nExample output:\nI/art (27274): AllocatorTagHeap active\u003d120 max\u003d120 total\u003d168\nI/art (27274): AllocatorTagMonitorList active\u003d1572 max\u003d6240 total\u003d11724\nI/art (27274): AllocatorTagClassTable active\u003d185208 max\u003d185208 total\u003d268608\nI/art (27274): AllocatorTagInternTable active\u003d430368 max\u003d430368 total\u003d436080\nI/art (27274): AllocatorTagMaps active\u003d5616 max\u003d6168 total\u003d34392\nI/art (27274): AllocatorTagLOS active\u003d1024 max\u003d1536 total\u003d2044\nI/art (27274): AllocatorTagSafeMap active\u003d0 max\u003d51936 total\u003d533688\nI/art (27274): AllocatorTagLOSMaps active\u003d144 max\u003d1248 total\u003d5760\nI/art (27274): AllocatorTagReferenceTable active\u003d10944 max\u003d11840 total\u003d19136\nI/art (27274): AllocatorTagHeapBitmap active\u003d32 max\u003d40 total\u003d56\nI/art (27274): AllocatorTagHeapBitmapLOS active\u003d8 max\u003d8 total\u003d8\nI/art (27274): AllocatorTagVerifier active\u003d0 max\u003d18844 total\u003d1073156\nI/art (27274): AllocatorTagModUnionCardSet active\u003d5300 max\u003d5920 total\u003d56020\nI/art (27274): AllocatorTagModUnionReferenceArray active\u003d24864 max\u003d24864 total\u003d24864\nI/art (27274): AllocatorTagJNILibrarires active\u003d320 max\u003d320 total\u003d320\nI/art (27274): AllocatorTagOatFile active\u003d1400 max\u003d1400 total\u003d5852\n\nChange-Id: Ibb470ef2e9c9a24563bb46422d46a55799704d82\n\n(cherry picked from commit 5369c40f75fdcb1be7a7c06db212ce965c83a164)\n" }, { "commit": "e77493c7217efdd1a0ecef521a6845a13da0305b", "tree": "3055cb7aaea8b9edc498b2e209d74af36c32e0fd", "parents": [ "41cba7c66cbc441b00fca48dfb2501181b1f2a53" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 20 15:08:45 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 20 15:09:20 2014 -0700" }, "message": "Make common BitVector operations inline-able.\n\nChange-Id: Ie25de4fae56c6712539f04172c42e3eff57df7ca\n" }, { "commit": "1ff3c98775a4577cf053dba9a0c2d5c21c07b298", "tree": "2d09c27c69678b53b4c9dc486024f3547efd4bca", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 02:30:58 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 16:08:05 2014 -0700" }, "message": "Avoid use of std::string where we have const char*.\n\nRemoving the ClassHelper caused std::string creation for all calls to\nClass::GetDescriptor and a significant performance regression. Make the\nstd::string an out argument so the caller can maintain it and its life time\nwhile allowing GetDescriptor to return the common const char* case.\n\nDon\u0027t generate GC maps when compilation is disabled.\n\nRemove other uses of std::string that are occuring on critical paths.\nUse the cheaper SkipClass in CompileMethod in CompilerDriver.\nSpecialize the utf8 as utf16 comparison code for the common shorter byte\nencoding.\nForce a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing\ncode), add some LIKELYs.\n\nx86-64 host 1-thread interpret-only of 57 apks:\nBefore: 29.539s\nAfter: 23.467s\n\nRegular compile:\nBefore: 1m35.347s\nAfter: 1m20.056s\n\nBug: 16853450\nChange-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad\n" }, { "commit": "68d8b42ddec39ec0174162d90d4abaa004d1983e", "tree": "0bb6ccf3b996bb8363c10d07aa39cde221980602", "parents": [ "9c522c2cbbf50dc687728747b37ad59985750b65" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:09:10 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 08 08:29:53 2014 -0700" }, "message": "Wire up check JNI force copy mode.\n\nIncrease check JNI checks.\nBreak apart jni_internal.h in to jni_env_ext.h and java_vm_ext.h.\nFix the abuse of ScopedObjectAccess/annotalysis by ScopedCheck in the case\nof VM routines.\nMake class loader override and shared library class loader JNI global\nreferences rather than mirror pointers.\nClean-ups to native bridge.\n\nChange-Id: If7c6110b5aade7a402bfb67534af86a7b2cdeb55\n" }, { "commit": "3f5838d7d0b9fc63db0ccc35c2ea05ed29264986", "tree": "b0f050237092e5bd171e7644e482bfa245f86602", "parents": [ "f4009e2249172ae8fe0e772ac8c0107400a28d6d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Aug 07 18:07:18 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 08 13:38:42 2014 +0100" }, "message": "Fix performance regression in OatFile::GetOatDexFile().\n\nTry to avoid calculating the canonical location of the\ndex file if possible and when we have to calculate it,\ncache the lookup result for subsequent lookups.\n\nBug: 16828525\nBug: 16859671\nChange-Id: I0f03007ba5adf08656615900cf125075a3f2c541\n" }, { "commit": "7747c8de402f64e3009ca3bcccebddbb70f617ee", "tree": "491fb0e346d6e0821d209b9617f0abfac78aedb1", "parents": [ "57cd9d3bda952ee2c2cf977fb1e26a0a954e1bab" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 06 14:53:03 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 06 16:06:28 2014 -0700" }, "message": "ART: Remove test files after test\n\nThe unix_file tests should remove files and directories when they\nare done testing.\n\nBug: 16505797\nChange-Id: Iff6856f64ee42ee9818b4ac23a6de3fe7eec6eae\n" }, { "commit": "a59dd80f9f48cb750d329d4d4af2d99d72b484d1", "tree": "36958b15842205addaf6d2a13e40823eab47c8bc", "parents": [ "84568fdf08f8f476292996ad653b4453d2894d23" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Jul 02 16:28:08 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 22 08:24:14 2014 -0700" }, "message": "Runtime can now be set to require relocation\n\nAdd a pair of runtime flags -Xrelocate and -Xnorelocate that can force\nthe runtime to require that all files that are run are relocated, to\nprevent attacks based on the known art base address.\n\nAdd support for running patchoat on oat files compiled without an image.\n\nChange run-test to have new --prebuild and --relocate flags.\n\nBug: 15358152\n\nChange-Id: I91166c62dd1ab80e5cbcb7883a2cd0d56afca32d\n" }, { "commit": "f3d874c60ee3ada19ce26a5c4e532312b6f3a9e9", "tree": "de14ab8b610f43a2c2c4c02f4ac67d614919fde2", "parents": [ "031ddea20cb311dfdb3bd16a13750f9cb426b299" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 18:52:42 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 18:59:40 2014 -0700" }, "message": "Avoid race in single thread suspension.\n\nDon\u0027t allow more than one concurrent single thread suspension to avoid\npotential cycles and deadlocks where threads try to suspend each other.\nBug: 16364458, 16354227\n\nChange-Id: I907f1d5591a6aa5c241d37d6b4a34f968f98df77\n" }, { "commit": "37f3c968ecd04e77802fe17bb82dabc07de21ca1", "tree": "90699a532f8c6f964a342c0f68b2afa605d7a335", "parents": [ "1817981832a531580c83f7d71f745da4e06889c3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:25:30 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:25:30 2014 -0700" }, "message": "Avoid direct use of QuasiAtomic by using Atomic.\n\nChange-Id: I92d0f822cddb2085de7cbb4c44c3a429081892a2\n" }, { "commit": "479f131d4bd3829dd512312020808b05f5a591f1", "tree": "a826b9fe32a5d107263c4bdda6f6255a7c7cd301", "parents": [ "e66f6318635c49313f796e62170fa8f59340e926", "4c13a3ff475f206c4d0a86ee2595c45392fd942f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 16 19:55:08 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 16 16:25:15 2014 +0000" }, "message": "Merge \"Disable adding main and non moving spaces to immune region in GSS\"" }, { "commit": "4c13a3ff475f206c4d0a86ee2595c45392fd942f", "tree": "5a96dff1dd5952aa70c2f25de46ea89d93e9225b", "parents": [ "524e5e3815d2b16ee77beda9976b7ec3aa54aba6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jul 14 14:57:16 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 16 12:34:43 2014 -0700" }, "message": "Disable adding main and non moving spaces to immune region in GSS\n\nDisabled adding the main and non moving space to the immune region.\nThis will enable us to recycle bump pointer spaces for malloc space\n-\u003e malloc space compaction as well as collector transitions.\n\nAlso added logic for falling back to the non moving space, we may\ncopy objects there.\n\nRefactored mod union table logic into MarkReachableObjects.\n\nNo measurable performance benefit or regression.\n\nBug: 14059466\nBug: 16291259\n\nChange-Id: If663d9fdbde943b988173b7f6ac844e5f78a0327\n" }, { "commit": "e63db27db913f1a88e2095a1ee8239b2bb9124e8", "tree": "893dee6783bca6717259321a6e4ba029c9c123e2", "parents": [ "07b8441303ea82fca3cb85d71ecf8752d73cedd7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 15:36:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 17:07:49 2014 -0700" }, "message": "Break apart header files.\n\nCreate libart-gtest for common runtime and compiler gtest routines.\nRename CompilerCallbacksImpl that is quick compiler specific.\nRename trace clock source constants to not use the overloaded profiler term.\n\nChange-Id: I4aac4bdc7e7850c68335f81e59a390133b54e933\n" }, { "commit": "c4f72ec44660f804b595bfaf2b959f46fd2ff00d", "tree": "df8764e4ec382b4e4810b8d0309823a40ef47d5c", "parents": [ "af4cf5d72a2bf77f0e442bedb208f9227f262c89", "74240819ae09e29b2753ef38f4eb4be1c2762e2e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 10 07:54:06 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 09 14:45:53 2014 +0000" }, "message": "Merge \"Use memory chunks for monitors on LP64\"" }, { "commit": "74240819ae09e29b2753ef38f4eb4be1c2762e2e", "tree": "61e2d3aa7268ce49fe77715593896f59feb92fe6", "parents": [ "32710dd4a0232149002a5ae7bde1c640cdffd564" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Apr 17 10:35:09 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 10 00:51:04 2014 -0700" }, "message": "Use memory chunks for monitors on LP64\n\nMonitor IDs in lock words are only 30b. On a 32b system that works\nfine, as memory is usually aligned enough that shifting works out.\nOn 64b systems, the virtual memory space is too large for that.\nThis adds memory chunks into which we allocate the monitors so that\nwe have base_addr + offset and can use the offset as the monitor ID.\nTo allow for relatively compact but growable storage, we use a list\nof chunks.\n\nAdded a global lock for the monitor pool.\n\nChange-Id: I0e290c4914a2556e0b2eef9902422d7c4dcf536d\n" }, { "commit": "c7190697f8665e706f6ebb4ae36fa63c46a32cd5", "tree": "f9281913f17e953143acd6c1a0f34dd3074bad56", "parents": [ "bcb3b29095817ce8987d8310d4db87271f5114ad" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 08 23:50:26 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 10 01:37:08 2014 +0000" }, "message": "Remove legacy CAS implementations from mutex.\n\nRemoves the use of __sync_bool_compare_and_swap and android_atomic_cas and uses\nintention revealing atomic operations from art::Atomic (which will eventually\ngive way to std::atomic).\n\nChange-Id: Iea44e1923f6706ec04b5459fe25427282c189a7e\n" }, { "commit": "833a48501d560c9fa7fc78ef619888138c2d374f", "tree": "add308298a5486d44caddea120cc9200dd70c38a", "parents": [ "b849f6dd638fd1246724160cd5c01ab1a5ff33bd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 21 18:46:59 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 25 19:34:58 2014 -0700" }, "message": "ART: Native support for multidex\n\nNative support for zip files with multiple classesX.dex.\n\nWorks by explicitly looking for those files in ascending order. As\nthese files have no file system representation for themselves,\nintroduce synthetic dex locations: the name of the originating file\nplus a colon plus the name of the dex file, e.g., test.jar:classes2.dex.\n\nOpening a zip dex file will return all dex files in this way. This\nkeeps the changes to dex2oat minimal.\n\nTo hide multidex/synthetic names from the Java layer, let the handle\nof dalvik.system.DexFile refer to a vector of DexFile objects. When\nopening a location, test possible synthetic names and add them to the\nvector. Thus, the original multidex jar in the classpath will be\nassociated with all embedded dex files.\n\nChange-Id: I0de107e1369cbc94416c544aca3b17525c9eac8b\n" }, { "commit": "f5997b4d3f889569d5a2b724d83d764bfbb8d106", "tree": "f0d3b3890a604a8b3c4c87c0f408f86df041690e", "parents": [ "9658d24deaa9bd07781bfac860d2a6dd89066d55" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 20 10:37:54 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 24 13:45:05 2014 -0700" }, "message": "More advanced timing loggers.\n\nThe new timing loggers have lower overhead since they only push into\na vector. The new format has two types, a start timing and a stop\ntiming. You can thing of these as brackets associated with a\ntimestamp. It uses these to construct various statistics when needed,\nsuch as: Total time, exclusive time, and nesting depth.\n\nChanged PrettyDuration to have a default of 3 digits after the decimal\npoint.\n\nExaple of a GC dump with exclusive / total times and indenting:\nI/art (23546): GC iteration timing logger [Exclusive time] [Total time]\nI/art (23546): 0ms InitializePhase\nI/art (23546): 0.305ms/167.746ms MarkingPhase\nI/art (23546): 0ms BindBitmaps\nI/art (23546): 0ms FindDefaultSpaceBitmap\nI/art (23546): 0ms/1.709ms ProcessCards\nI/art (23546): 0.183ms ImageModUnionClearCards\nI/art (23546): 0.916ms ZygoteModUnionClearCards\nI/art (23546): 0.610ms AllocSpaceClearCards\nI/art (23546): 1.373ms AllocSpaceClearCards\nI/art (23546): 0.305ms/6.318ms MarkRoots\nI/art (23546): 2.106ms MarkRootsCheckpoint\nI/art (23546): 0.153ms MarkNonThreadRoots\nI/art (23546): 4.287ms MarkConcurrentRoots\nI/art (23546): 43.461ms UpdateAndMarkImageModUnionTable\nI/art (23546): 0ms/112.712ms RecursiveMark\nI/art (23546): 112.712ms ProcessMarkStack\nI/art (23546): 0.610ms/2.777ms PreCleanCards\nI/art (23546): 0.305ms/0.855ms ProcessCards\nI/art (23546): 0.153ms ImageModUnionClearCards\nI/art (23546): 0.610ms ZygoteModUnionClearCards\nI/art (23546): 0.610ms AllocSpaceClearCards\nI/art (23546): 0.549ms AllocSpaceClearCards\nI/art (23546): 0.549ms MarkRootsCheckpoint\nI/art (23546): 0.610ms MarkNonThreadRoots\nI/art (23546): 0ms MarkConcurrentRoots\nI/art (23546): 0.610ms ScanGrayImageSpaceObjects\nI/art (23546): 0.305ms ScanGrayZygoteSpaceObjects\nI/art (23546): 0.305ms ScanGrayAllocSpaceObjects\nI/art (23546): 1.129ms ScanGrayAllocSpaceObjects\nI/art (23546): 0ms ProcessMarkStack\nI/art (23546): 0ms/0.977ms (Paused)PausePhase\nI/art (23546): 0.244ms ReMarkRoots\nI/art (23546): 0.672ms (Paused)ScanGrayObjects\nI/art (23546): 0ms (Paused)ProcessMarkStack\nI/art (23546): 0ms/0.610ms SwapStacks\nI/art (23546): 0.610ms RevokeAllThreadLocalAllocationStacks\nI/art (23546): 0ms PreSweepingGcVerification\nI/art (23546): 0ms/10.621ms ReclaimPhase\nI/art (23546): 0.610ms/0.702ms ProcessReferences\nI/art (23546): 0.214ms/0.641ms EnqueueFinalizerReferences\nI/art (23546): 0.427ms ProcessMarkStack\nI/art (23546): 0.488ms SweepSystemWeaks\nI/art (23546): 0.824ms/9.400ms Sweep\nI/art (23546): 0ms SweepMallocSpace\nI/art (23546): 0.214ms SweepZygoteSpace\nI/art (23546): 0.122ms SweepMallocSpace\nI/art (23546): 6.226ms SweepMallocSpace\nI/art (23546): 0ms SweepMallocSpace\nI/art (23546): 2.144ms SweepLargeObjects\nI/art (23546): 0.305ms SwapBitmaps\nI/art (23546): 0ms UnBindBitmaps\nI/art (23546): 0.275ms FinishPhase\nI/art (23546): GC iteration timing logger: end, 178.971ms\n\nChange-Id: Ia55b65609468f212b3cd65cda66b843da42be645\n" }, { "commit": "825201e9a7e718470e2dac222dffdee729050ac5", "tree": "417dd2ac4002563797d2e579b2c28cb4e8089a02", "parents": [ "2d2d6bb1ff8692efdddf104dabfb4a2d39877ff6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 20 10:45:30 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jun 20 10:45:30 2014 -0700" }, "message": "ART: ReadFully needs to fail when EOF\n\nBesides an explicit error, we should stop trying when we unexpectedly\nreach the end of the file.\n\nChange-Id: I16d601e1b5bcbc39cb8c4dd7fd3dbb7d69016579\n" }, { "commit": "0c29909cbde112bc9c04da4ce81421e1a0b39f36", "tree": "a925100d9eed7aaeee7f3d0fa16a8917f6370c12", "parents": [ "241fd1192dfc0f7322660343179f9fc0591ed9ff", "7de77dd4f2d3cbb0615ee001589eb99ae82c3dcc" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jun 18 23:24:40 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 18 22:48:23 2014 +0000" }, "message": "Merge \"Assign a lower Locklevel for Mutexes used in QuasiAtomic operations. This fixes the CompilerDriverTest on MIPS. The test was failing when Transaction::Abort() acquired intern_table_lock_ and log_lock_ and subsequently performed QuasiAtomic operations.\"" }, { "commit": "19d46b44f2abe742be22e32908dbfd9e6dd9bfea", "tree": "c0027b328a57280afaad197189a3d58c73242bb4", "parents": [ "4fab5a1f9b34f558d1f4002de73e233d8a0d47b7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 17 15:04:40 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 17 15:43:14 2014 -0700" }, "message": "Fix systrace logging, total paused time, and bytes saved message.\n\nMoved the GC top level systrace logging to be inside of Collector::Run.\nThis prevents cases where we forgot to call it such as background\ncompaction. Fixed a unit error regarding total pause time. Fixed\nnegative bytes saved to use the word \"expanded\".\n\nBug: 15702709\n\nChange-Id: Ic2991ecad2daa000d0aee9d559b8bc77d8c160aa\n" }, { "commit": "7de77dd4f2d3cbb0615ee001589eb99ae82c3dcc", "tree": "b699dc76d48c853897d8cabaa825be2ab4ec4f5d", "parents": [ "8cd927c8f6911cfb6118b8c54550959c09eb84f8" ], "author": { "name": "Raghu Gandham", "email": "raghu.gandham@imgtec.com", "time": "Fri Jun 13 15:16:31 2014 -0700" }, "committer": { "name": "Raghu Gandham", "email": "raghu.gandham@imgtec.com", "time": "Fri Jun 13 15:26:39 2014 -0700" }, "message": "Assign a lower Locklevel for Mutexes used in QuasiAtomic operations.\nThis fixes the CompilerDriverTest on MIPS. The test was failing when\nTransaction::Abort() acquired intern_table_lock_ and log_lock_\nand subsequently performed QuasiAtomic operations.\n\nChange-Id: I9fcda51221f1f298bcb5dd0e10019bd2034ab8ae\n" }, { "commit": "3e1e549c564045d852ace46388eb06427d63e6ca", "tree": "810bf5bc158a872eec2bfbb2a7a3e3a0dd59c9a6", "parents": [ "dfc2091d2fb8a7694f69acf8bd39ce4953e026c2", "836424805dd48af67088d30992a3f2e6841ca047" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 12 14:27:31 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 11 15:59:20 2014 +0000" }, "message": "Merge \"Add D/CHECK_CONSTEXPR macros for use in constexpr context.\"" }, { "commit": "836424805dd48af67088d30992a3f2e6841ca047", "tree": "7ef4b2fbe4ec82f3845f201f1a3befe70571739d", "parents": [ "3283ade469f515f89d70cf47b6ac6fe1814147f2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 11 12:12:07 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 11 14:07:28 2014 +0100" }, "message": "Add D/CHECK_CONSTEXPR macros for use in constexpr context.\n\nMake most RegStorage methods constexpr, make StorageSize()\nconst.\n\nChange-Id: Ie63faa3b081094b3dd30352c6240fdf10ef8b87e\n" }, { "commit": "c5f17732d8144491c642776b6b48c85dfadf4b52", "tree": "811daa488ae5ee5dfd9b3b73bd210bc1506e5ca1", "parents": [ "08654d40cdd256f6a6c8619bf06d04d4c819714a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jun 05 20:48:42 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 10 23:19:29 2014 -0700" }, "message": "Remove deprecated WITH_HOST_DALVIK.\n\nBug: 13751317\nFix the Mac build:\n - disable x86 selector removal that causes OS/X 10.9 kernel panics,\n - madvise don\u0027t need does zero memory on the Mac, factor into MemMap\n routine,\n - switch to the elf.h in elfutils to avoid Linux kernel dependencies,\n - we can\u0027t rely on exclusive_owner_ being available from other pthread\n libraries so maintain our own when futexes aren\u0027t available (we\n can\u0027t rely on the OS/X 10.8 hack any more),\n - fix symbol naming in assembly code,\n - work around C library differences,\n - disable backtrace in DumpNativeStack to avoid a broken libbacktrace\n dependency,\n - disable main thread signal handling logic,\n - align the stack in stub_test,\n - use $(HOST_SHLIB_SUFFIX) rather than .so in host make file variables.\n\nNot all host tests are passing on the Mac with this change. dex2oat\nworks as does running HelloWorld.\nChange-Id: I5a232aedfb2028524d49daa6397a8e60f3ee40d3\n" }, { "commit": "d1c606f280797be81e2592c483869a6ec836a9f3", "tree": "f49689281100d0fccdc5493f1a2567b435a247ea", "parents": [ "fbc2b1747e7e3d06f214f801f53218a1d4bf2dbe" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Jun 09 16:50:19 2014 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Jun 10 11:25:06 2014 +0100" }, "message": "Add locking around boot image generation.\n\nIf zygote aborts due to an error, it will restart and\nspawn another dex2oat process while the old one is still\nrunning. If this happens fast enough, the system will\neventually need a kernel reboot since neither the zygote\nnor dex2oat are killable.\n\nThis brings boot image generation in line with dex2oat\ngeneration, which uses a similar pattern of advisory\nlocking.\n\nbug: 15415316\n\nChange-Id: Iaccd274d3d96ab002b04e246ec4b3ef9a422ff7c\n" }, { "commit": "c2bb441a65c5d1dd0b73d04489514ab09bd53c8a", "tree": "a19fbf8372a40441812facb27ec0f661bf395f7c", "parents": [ "4479ba35389b03ccc9eabd17fba6168f9505517a", "3035961cb41865b80b927546be0c708b6389cec6" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jun 06 23:45:51 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 06 23:45:52 2014 +0000" }, "message": "Merge \"Clean up and augment Atomic class. Replace QuasiAtomic MemBars.\"" }, { "commit": "576ca0cd692c0b6ae70e776de91015b8ff000a08", "tree": "de632fae7ee9f096e7a2c54a7fcdace8f6ae7106", "parents": [ "e21bf0099117b82963cdf7f95a1b8dabfcf29397" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 06 15:58:22 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 06 16:27:45 2014 -0700" }, "message": "Reduce header files including header files.\n\nMain focus is getting heap.h out of runtime.h.\n\nChange-Id: I8d13dce8512816db2820a27b24f5866cc871a04b\n" }, { "commit": "3035961cb41865b80b927546be0c708b6389cec6", "tree": "27ed39f92c013381d08c7ad51d311cb29be230dc", "parents": [ "e21bf0099117b82963cdf7f95a1b8dabfcf29397" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed May 21 17:46:23 2014 -0700" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jun 06 16:13:42 2014 -0700" }, "message": "Clean up and augment Atomic class. Replace QuasiAtomic MemBars.\n\nAdd a number of missing C++11 operations to Atomic class.\nInvoke the 64 bit routines in QuasiAtomic when necessary.\nReplace QuasiAtomic membars with fences that correspond to C++11 fences.\n\nQuasiAtomic was moved to the top of the file. Only fence implementations\nactually changed.\n\nThis replaces some buggy uses of MembarStoreStore, as reported\nin b/14685856 .\n\nAvoid some redundant fences for long volatile operations.\n\nIncompletely converts low-level memory access operations to Atomic.\n\nChange-Id: Iea828431a0cea46540eb74fcaa02071cab6fdcda\n" }, { "commit": "7ce610516242b1ffd47a42bfc31c2d562f443ca6", "tree": "500f4d03a882aebbefa33e3b29be0a2b0518ad48", "parents": [ "d5e2b00cbc1e6b6608a5dbcfb05b7c4976614636", "3eed93dd5be03e5539827bebf0f414251a12e15e" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jun 05 18:48:54 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jun 05 18:48:54 2014 +0000" }, "message": "Merge \"Verify there\u0027s no mem map gap for immune region not to break.\"" }, { "commit": "3eed93dd5be03e5539827bebf0f414251a12e15e", "tree": "186fa74420e2595c9b6fba9ea8de94756af9cef5", "parents": [ "bbdc5bc5fd5141711879a6c85d80ac45b7aad5d0" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Jun 04 11:43:59 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Jun 04 15:50:46 2014 -0700" }, "message": "Verify there\u0027s no mem map gap for immune region not to break.\n\nThis adds code that verifies that there\u0027s no memory map gap between\nthe image space and the main space so that the immune region\nfunctionality won\u0027t silently break. For example, if there\u0027s a gap and\na large object is allocated in that gap, the large object is\nincorrectly part of the immune region and the marking breaks.\n\nBug: 14059466\nChange-Id: Ie6ed82988d74b6d0562ebbbaac96ee43c15b14a6\n" }, { "commit": "903e390ffd46955628d1161ccfa8de9177d42d22", "tree": "d828cc537e0accb55674ebcc3908c5d713bb7271", "parents": [ "955bc573c5611af6a31d9dd7cd93c210a7ad107a", "47a4cc7ad629d499b9f4841829d0371728075433" ], "author": { "name": "Mark Salyzyn", "email": "salyzyn@google.com", "time": "Wed Jun 04 19:27:12 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 04 19:27:13 2014 +0000" }, "message": "Merge \"art: -Wsystem-header warning\"" }, { "commit": "014d77a2107fec8ba978a7428fd4d04e0bf8e168", "tree": "af30717fb54eedf6af9d931cb89656f29f701134", "parents": [ "e4283be97047a26d3476acd3863dcc386498be17" ], "author": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Mon Jun 02 11:21:21 2014 -0700" }, "committer": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Mon Jun 02 15:17:03 2014 -0700" }, "message": "ART: BitVector not calculating number_of_bits correctly\n\nThe number_of_bits_ field has an unclear intent. Instead, using\n storage_size_ * kWordBits when relevant.\n\nChange-Id: I8c13be0d6643de37813fb154296d451f22c298c8\nSigned-off-by: Jean Christophe Beyler \u003cjean.christophe.beyler@intel.com\u003e\n" }, { "commit": "7f0a6d6a699111e254a91c3980eb0522d4f8f0cd", "tree": "1cbd1b5c2f4d61b73eeb1a2e780235f6ca70fc68", "parents": [ "ddb1a1604d5070828e857902c8596de27e2fe5f5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 26 14:01:46 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 26 14:01:46 2014 +0100" }, "message": "Fix x86_64 test failures.\n\nChange-Id: I97b3d1755d2e205ca7423248a2af37f1875d5116\n" }, { "commit": "06abcdfa518d1fe3cf87416f5bca6af1d0e2d569", "tree": "522a967761a9c37cce23a6d0e53eef6f54283f9e", "parents": [ "82f3c1b76a19f196a482cad9d5c1aac769d5a397" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 23 11:39:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 23 11:39:11 2014 -0700" }, "message": "Extra debug output for a failing lock check.\n\nChange-Id: If4abae535a875a06f4fa4203e6aabc1169b1f38e\n" }, { "commit": "ec23021ceb9f91a424c524fbfbcc9de6fb21f2b1", "tree": "f357f025008f6b3bac1c4881ee77ff4999bbd951", "parents": [ "ccddc49852a69094aecf9eb7c7de67a9c2c5c31a", "520f37bb5c34c5d86ad0091cb84a84c163a2fa9c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 23 18:03:11 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri May 23 18:03:11 2014 +0000" }, "message": "Merge \"ART: Added print indices back to BitVector Dumper\"" }, { "commit": "920be0b27c72ceb3d40b5f2775cd1950f7c65b5f", "tree": "e1ebbec3ec4b2a73d08f798a318ecde9be888ede", "parents": [ "e24580a135aa6e85cdc49426ec74d2ae1e326243" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 23 18:43:51 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 23 18:43:51 2014 +0100" }, "message": "Fix style issue.\n\nChange-Id: I2044e01c68265c33e7fa6057efa7b6c7ac41ada4\n" }, { "commit": "520f37bb5c34c5d86ad0091cb84a84c163a2fa9c", "tree": "8c1ebd76a690dfa328821cd26b78d48531848cfa", "parents": [ "b8033db2a8dc6f7c7e29b1552177542964f56e44" ], "author": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Thu May 22 15:43:50 2014 -0700" }, "committer": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Fri May 23 09:31:27 2014 -0700" }, "message": "ART: Added print indices back to BitVector Dumper\n\n- Added an API to get the indices set instead of 001...0 format\n\nChange-Id: I75841e41ca9b7ef77a0717715669dbe12506d6a1\nSigned-Off-By: Jean Christophe Beyler \u003cjean.christophe.beyler@intel.com\u003e\n" }, { "commit": "a5b8fde2d2bc3167078694fad417fddfe442a6fd", "tree": "287942554467eb8566291f7d021549f65763f53e", "parents": [ "567e9dbc65ee183cda2a052dbf224c8c4a8f9423" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 23 15:16:44 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 23 17:12:15 2014 +0100" }, "message": "Rewrite BitVector index iterator.\n\nThe BitVector::Iterator was not iterating over the bits but\nrather over indexes of the set bits. Therefore, we rename it\nto IndexIterator and provide a BitVector::Indexes() to get\na container-style interface with begin() and end() for range\nbased for loops.\n\nAlso, simplify InsertPhiNodes where the tmp_blocks isn\u0027t\nneeded since the phi_nodes and input_blocks cannot lose any\nblocks in subsequent iterations, so we can do the Union()\ndirectly in those bit vectors and we need to repeat the loop\nonly if we have new input_blocks, rather than on phi_nodes\nchange. And move the temporary bit vectors to scoped arena.\n\nChange-Id: I6cb87a2f60724eeef67c6aaa34b36ed5acde6d43\n" }, { "commit": "47a4cc7ad629d499b9f4841829d0371728075433", "tree": "e0e79df7fa34a6adcd16070c30009c79fe991186", "parents": [ "be2cb9b284f3eca3b99571e83b7e0e3109f1ef06" ], "author": { "name": "Mark Salyzyn", "email": "salyzyn@google.com", "time": "Thu May 22 16:27:06 2014 -0700" }, "committer": { "name": "Mark Salyzyn", "email": "salyzyn@google.com", "time": "Thu May 22 17:10:54 2014 -0700" }, "message": "art: -Wsystem-header warning\n\n- TEMP_FAILURE_RETRY redefined if unistd.h is include later\n\nChange-Id: I79101ebfbec47e7854414c4ef40a91ef122ae284\n" }, { "commit": "9e36931cc79ca665908db9575126881d1cfdea5a", "tree": "69099016576581e3e4d6c44ee01a7a6cc7db4b2c", "parents": [ "5da6e2b88258733dd3856543af27ca73b395ef60" ], "author": { "name": "Chao-ying Fu", "email": "chao-ying.fu@intel.com", "time": "Wed May 21 11:20:52 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 22 10:25:57 2014 -0700" }, "message": "Move modify_ldt_lock into global lock order.\n\nMutex modify_ldt_lock was being removed during runtime shutdown while\ndaemons thread may still detach. Avoid this by placing in global lock\norder.\nThis fixes cts dalvik vm-tests-tf that hang on some x86 devices.\n\nBy irogers: also, tidy global locks to agree with enum constants and\nadd extra verification that the global annotalysis order agrees with\nthe LockLevel order. Bumped the oat version and moved the locks as\nLockLevel additions previously caused entrypoints to be moved. Make\nunattached lock not handle the default mutex level case by moving the\nallocated thread ids lock into the global order.\n\nChange-Id: I9d03f19d44ea254accf0ceae8022563c77f7a02f\nSigned-off-by: Chao-ying Fu \u003cchao-ying.fu@intel.com\u003e\n" }, { "commit": "4996f1191cd3d5edf85981d6412aa129da2dcd06", "tree": "eac9ee01c79388275c17ae53e0ddebaa1d4b618b", "parents": [ "aca0ca9d949dee1bb4fe741b03a0514b324cc035", "57f80fa56effba065a3a71847e9a0d07164aca14" ], "author": { "name": "Dan Albert", "email": "danalbert@google.com", "time": "Thu May 22 03:50:35 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu May 22 03:50:35 2014 +0000" }, "message": "Merge \"Use decltype() instead of typeof()\"" }, { "commit": "57f80fa56effba065a3a71847e9a0d07164aca14", "tree": "92edf48567d2bd9f07bf731d845f1af3fc884a76", "parents": [ "9c47134233b3f191690eb142141de95aeeb62d2a" ], "author": { "name": "Dan Albert", "email": "danalbert@google.com", "time": "Wed May 21 20:31:55 2014 -0700" }, "committer": { "name": "Dan Albert", "email": "danalbert@google.com", "time": "Wed May 21 20:31:55 2014 -0700" }, "message": "Use decltype() instead of typeof()\n\nThis fixes a build failure on Mac.\n\nChange-Id: Iee73ba990d2494fd925fd237da68cf753709a248\n" }, { "commit": "3e5cf305db800b2989ad57b7cde8fb3cc9fa1b9e", "tree": "52a737323ebd505cf37ca0e4b2dcee6524fba07f", "parents": [ "27a2b70f612af9afc0fb5392fb10059f6a0a3569" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue May 20 16:40:37 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue May 20 21:17:03 2014 -0700" }, "message": "Begin migration of art::Atomic to std::atomic.\n\nChange-Id: I4858d9cbed95e5ca560956b9dabd976cebe68333\n" }, { "commit": "700a402244a1a423da4f3ba8032459f4b65fa18f", "tree": "4c22fcda04d271bd55a37aff30650214af17a90c", "parents": [ "047c11adcbcbc0bcf210defdfcbada763961ffee" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 19 16:49:03 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 19 22:27:39 2014 -0700" }, "message": "Now we have a proper C++ library, use std::unique_ptr.\n\nAlso remove the Android.libcxx.mk and other bits of stlport compatibility\nmechanics.\n\nChange-Id: Icdf7188ba3c79cdf5617672c1cfd0a68ae596a61\n" }, { "commit": "507dfdd147c97bfbadebfd63584d094b6a4e7b47", "tree": "cce43931b6dcd088cb2932c2491f86116353a27f", "parents": [ "922ddb30982d2597eab634d8b8598bec0eb7d3b7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 15 16:42:40 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 15 16:50:51 2014 -0700" }, "message": "Compatibility layer to transition from UniquePtr to std::unique_ptr.\n\nUse ART_WITH_STLPORT (enabled for the target) to cause the use of UniquePtr,\nfor the host switch to std::unique_ptr. For now the type remains called\nUniquePtr.\nMake dalvik compile with clang on the host, move its build to C++11.\n\nChange-Id: I5ba8d2757904bc089ed62047ea03de3c0853fb12\n" }, { "commit": "622d9c31febd950255b36a48b47e1f630197c5fe", "tree": "8a7f14ce3c6c087955ad5fe91a3ce7d5b5a82461", "parents": [ "98a8a542f95e41c09d214a329a940b270f08f5b3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 12 16:11:02 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue May 13 09:06:14 2014 +0100" }, "message": "Add loop recognition and CFG simplifications in new compiler.\n\nWe do three simplifications:\n- Split critical edges, for code generation from SSA (new).\n- Ensure one back edge per loop, to simplify loop recognition (new).\n- Ensure only one pre header for a loop, to simplify SSA creation (existing).\n\nChange-Id: I9bfccd4b236a00486a261078627b091c8a68be33\n" }, { "commit": "4d466a8e4587422c989705dce3b2a19e7f0137f5", "tree": "bd8f0f45db9a7d5888b14a070690f59fb599bd68", "parents": [ "f1c036fc8fa97617cc42bda44a0adbc7b856d6cd" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu May 08 19:05:29 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri May 09 12:23:00 2014 -0700" }, "message": "Remove all LOG(DEBUG) which is not intended to be checked in\n\nBug: 14632493\nChange-Id: Id9da8b87798af0a1b2bd2c178133e5f1ba47d43d\n" }, { "commit": "2d1ab0a7bf7639d1af0c453415f9625110c34f6d", "tree": "765ff7d49c4a9016bfbdf92a43f2aed2c0d0f4d3", "parents": [ "4884ba0c06fbd167a29a057787f4f35160d2ebd1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 08 15:27:31 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 08 16:59:29 2014 -0700" }, "message": "Use WaitHoldingLocks in ReferenceProcessor::GetReferent.\n\nThe caller may be holding other locks, so we must use this or else\nwe get bad mutexes held errors. Also added a lock level for the\nreference processor.\n\nChange-Id: I364a303a4106982bd9bfaac5b116cea1638ea6d3\n" }, { "commit": "221b86d96f6e1971d24e3d6a283352c58cedbd32", "tree": "f80439dc10bf971004472a7aa609aa0a9cff65eb", "parents": [ "60280e50c1ef4a7ef286670975a81f8384138f42", "b5c9b4008760c9042061490f22aaff990ed04c9a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 07 15:15:46 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 07 15:15:46 2014 +0000" }, "message": "Merge \"ART: BitVector and Optimization changes\"" }, { "commit": "f23a719218ffaa37a85482e40509e62275554774", "tree": "d04b4d12264758169a7d6c8ede2427f1078249f3", "parents": [ "290fda3acf3d9ce60f7ac3903fabfb01d3521d0f", "804d09372cc3d80d537da1489da4a45e0e19aa5d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 07 09:33:54 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 07 09:33:54 2014 +0000" }, "message": "Merge \"Build live-in, live-out and kill sets for each block.\"" }, { "commit": "804d09372cc3d80d537da1489da4a45e0e19aa5d", "tree": "b226350fdf3dc0c55a11e1615010c8475f167f90", "parents": [ "0095e0b8380a8802f40a21928800b9df6e11f1d7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 02 08:46:00 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 07 10:32:11 2014 +0100" }, "message": "Build live-in, live-out and kill sets for each block.\n\nThis information will be used when computing live ranges of\ninstructions.\n\nChange-Id: I345ee833c1ccb4a8e725c7976453f6d58d350d74\n" }, { "commit": "b5c9b4008760c9042061490f22aaff990ed04c9a", "tree": "c3fc539b056dadcad2c47c9899caeecc5f2d00ae", "parents": [ "36b65964d128471d917c2efc69c81bc50ef9360b" ], "author": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Wed Apr 30 14:52:00 2014 -0700" }, "committer": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Tue May 06 13:47:51 2014 -0700" }, "message": "ART: BitVector and Optimization changes\n\n- The BitVector has a function SameBitsSet that is a bit upside down\n - This patch fixes it.\n\n- Two optimizations are fixed also:\n - The null check pass uses now same bits set instead of equal due to a\n subsequent change that will make it not always the case that the\n compared bitvectors be of the same size.\n - The fused optimization supposes a predecessor will have an instruction.\n\nChange-Id: I9ef1c793964b18dc0f47baf9d1f361448bb053a3\nSigned-off-by: Jean Christophe Beyler \u003cjean.christophe.beyler@intel.com\u003e\nSigned-off-by: Razvan A Lupusoru \u003crazvan.a.lupusoru@intel.com\u003e\nSigned-off-by: Yixin Shou \u003cyixin.shou@intel.com\u003e\nSigned-off-by: Chao-ying Fu \u003cchao-ying.fu@intel.com\u003e\nSigned-off-by: Udayan Banerji \u003cudayan.banerji@intel.com\u003e\n" }, { "commit": "5cd33753b96d92c03e3cb10cb802e68fb6ef2f21", "tree": "b2217cb48e1f1db1aae7a92f8d5dbac5f2d95603", "parents": [ "8ea5baa10b04e9dbd0f0cf14b0358fe8b956bb2e" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Tue Apr 15 15:57:58 2014 -0700" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Mon May 05 12:44:15 2014 -0700" }, "message": "Handle implicit stack overflow without affecting stack walks\n\nThis changes the way in which implicit stack overflows are handled\nto satisfy concerns about changes to the stack walk code.\n\nInstead of creating a gap in the stack and checking for it in\nthe stack walker, use the ManagedStack infrastructure to concoct\nan invisible gap that will never be seen by a stack walk.\n\nAlso, this uses madvise to tell the kernel that the main stack\u0027s\nprotected region will probably never be accessed, and instead\nof using memset to map the pages in, use memcpy to read from\nthem. This will save 32K on the main stack.\n\nAlso adds a \u0027signals\u0027 verbosity level as per a review request.\n\nBug: 14066862\nChange-Id: I5257305feeaea241d11e6aa6f021d2a81da20b81\n" }, { "commit": "8194963098247be6bca9cc4a54dbfa65c73e8ccc", "tree": "547cc708e06e6541676b17066023ae6f07b2049b", "parents": [ "56a341a82ece9aa4f2a071629f3e1fd1adf988ae" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 02 11:53:22 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 02 12:21:02 2014 +0100" }, "message": "Replace CountOneBits and __builtin_popcount with POPCOUNT.\n\nClean up utils.h, make some functions constexpr.\n\nChange-Id: I2399100280cbce81c3c4f5765f0680c1ddcb5883\n" }, { "commit": "6d2387765ed0a898281d126bc3d4f9b124667d5d", "tree": "63822d72bb1e4d326b2c8f82df5541fd3833f04d", "parents": [ "990d46f3333dce06a640ced697ee069330a73d7c", "5afa08f95d43dd24fb4b3d7a08aa1ec23386ad54" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 28 10:18:30 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Apr 28 10:18:30 2014 +0000" }, "message": "Merge \"ART: Bitvector extensions for dumping and size handling\"" }, { "commit": "73d1e17b3afc7d5e56184f90bf819dc64956448a", "tree": "6fee546dcebc0c8979a62be7e75c337f79b2ac2e", "parents": [ "e8256e7773a230337c3d137cbf0365f737820405" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 11 17:53:48 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 25 14:08:49 2014 -0700" }, "message": "Enable reading page map without lock in RosAlloc::BulkFree\n\nEnabling this flag greatly reduces how much time was spent in the GC.\nIt was not done previously since it was regressing MemAllocTest. With\nthese RosAlloc changes, the benchmark score no longer regresses after\nwe enable the flag.\n\nChanged Run::AllocSlot to only have one mode of allocation. The new\nmode is finding the first free bit in the bitmap. This was\npreviously the slow path but is now the fast path. Some optimizations\nwhich enabled this include always having the alloc bitmap bits which\ncorrespond to invalid slots be set to 1. This prevents us from needing\na bound check since we will never end up allocating there.\n\nChanged revoking thread local buffer to point to an invalid run. The\ninvalid run is just a run which always has all the allocation bits set\nto 1. When a thread attempts to do a thread local allocation from here\nit will always fail and go slow path. This eliminates the need for a\nnull check for revoked runs.\n\nChanged zeroing of memory to happen during free, AllocPages should\nalways return zeroed memory. Added prefetching which happens when we\nallocate a run.\n\nSome refactoring to reduce duplicated code.\n\nErgonomics changes: Changed kStickyGcThroughputAdjustment to 1.0,\nthis helps reduce GC time.\n\nMeasurements (3 samples per benchmark):\nBefore: MemAllocTest scores: 3463, 3445, 3431\nEvaluateAndApplyChanges score | total GC time\nIter 1: 3485, 23.602436s\nIter 2: 3434, 22.499882s\nIter 3: 3483, 23.253274s\n\nAfter: MemAllocTest scores: 3495, 3417, 3409\nEvaluateAndApplyChanges score | total GC time:\nIter 1: 3375, 17.463462s\nIter 2: 3358, 16.185188s\nIter 3: 3367, 15.822312s\n\nBug: 8788501\nBug: 11790317\nBug: 9986565\nChange-Id: Ifd273a054824028dabed27c07c081dde1816f93c\n" }, { "commit": "5afa08f95d43dd24fb4b3d7a08aa1ec23386ad54", "tree": "28d388989be9ccdfce387728f63820689c29e9ca", "parents": [ "1c17753ca9cf125ffd1fa47f34650fb6f4005409" ], "author": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Tue Apr 15 15:54:35 2014 -0700" }, "committer": { "name": "Jean Christophe Beyler", "email": "jean.christophe.beyler@intel.com", "time": "Thu Apr 24 12:20:40 2014 -0700" }, "message": "ART: Bitvector extensions for dumping and size handling\n\n- Added dumping functions and the ensure size and clear function.\n- Fixed a bug in union where if a bit is not set in the source, an buffer\n overflow can occur later down.\n\nChange-Id: Iff40529f3a8970a1ce2dd5c591f659f71924dea3\nSigned-off-by: Jean Christophe Beyler \u003cjean.christophe.beyler@intel.com\u003e\nSigned-off-by: Razvan A Lupusoru \u003crazvan.a.lupusoru@intel.com\u003e\nSigned-off-by: Yixin Shou \u003cyixin.shou@intel.com\u003e\nSigned-off-by: Chao-ying Fu \u003cchao-ying.fu@intel.com\u003e\nSigned-off-by: Udayan Banerji \u003cudayan.banerji@intel.com\u003e\n" }, { "commit": "82b1a81890970a8b07f9132aeae537a6c43df6b0", "tree": "43ac558ccae56799d093e58c128af83511889f5f", "parents": [ "618a87009202dc959c935ed8f237ae32bdec57d0", "d3c5bebcb52a67cb06e7ab303eaf45f230c08b60" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 22 13:25:20 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 22 13:25:20 2014 +0000" }, "message": "Merge \"Avoid allocating OatFile::OatClass on the heap.\"" }, { "commit": "42d65c52bbf4a354605389a759190e793787f692", "tree": "98dd633038cbc1ae0ccdda3de1f73e79a9ab4049", "parents": [ "fa25af3e3410537e86b485682d519dafbad01753" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Fri Apr 18 16:49:39 2014 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Fri Apr 18 16:51:15 2014 -0700" }, "message": "Add an option to specify a list of methods for LIR dumping.\n\nAn example would be:\n--runtime-arg -verbose-methods:Array.checkRange32,Array.checkRange64\nor\n\"-verbose-methods:void Array.checkRange32,void Array.checkRange64\"\n\nChange-Id: I61c1eb3b2eb4b24126a9264261c27889d53cc6bd\n" }, { "commit": "01bc96d007b67fdb7fe349232a83e4b354ce3d08", "tree": "1ed36c2d7c0fb204e6f276bd9853153d9350ad1d", "parents": [ "2be6fc74bce10ac68d3d1b39a5019f520ad170ea" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 11 17:43:50 2014 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Apr 16 14:21:12 2014 +0100" }, "message": "Long support in optimizing compiler.\n\n- Add stack locations to the Location class.\n- Change logic of parameter passing/setup by setting the\n location of such instructions the ones for the calling\n convention.\n\nChange-Id: I4730ad58732813dcb9c238f44f55dfc0baa18799\n" }, { "commit": "d3c5bebcb52a67cb06e7ab303eaf45f230c08b60", "tree": "85df477ba745b1e2c85ab1d167b0297236060f2f", "parents": [ "043a7a6182870964021476484b1534106ff20600" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 11 16:32:51 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 14 18:03:41 2014 +0100" }, "message": "Avoid allocating OatFile::OatClass on the heap.\n\nAvoid allocating a BitVector for OatFile::OatClass::bitmap_\nwith kOatClassSomeCompiled methods. That makes the OatClass\ncopy-constructible as it doesn\u0027t own any memory. We use that\nin OatFile::OatDexFile::GetOatClass() to return the result\nby value thus avoiding one or two heap allocations per call.\n\nChange-Id: Ic7098109028a5b49e39ef626f877de86e732ed18\n" }, { "commit": "440e4ceb310349ee8eb569495bc04d3d7fbe71cb", "tree": "db278d463f0c2440e9c8e8cb80e53d2b2effe196", "parents": [ "4d3dc8a04da8a80251b381cdb482190363171dc1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 31 16:36:35 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 07 13:44:36 2014 -0700" }, "message": "Add monitor deflation.\n\nWe now deflate the monitors when we perform a heap trim. This causes\na pause but it shouldn\u0027t matter since we should be in a state where\nwe don\u0027t care about pauses. Memory savings are hard to measure.\n\nFixed integer overflow bug in GetEstimatedLastIterationThroughput.\n\nBug: 13733906\nChange-Id: I4e0e68add02e7f43370b3a5ea763d6fe8a5b212c\n" }, { "commit": "8732bf96433bbc177de509390d285b2d5477c83d", "tree": "3c8974efdff51311910e155b226a7e64060c7e7b", "parents": [ "7f466c08888129a9923cb973a4dc73ee4a71574e", "51b71028661092e8860cca4f8ca79848e03cdc2c" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Mar 31 10:03:07 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Mar 31 10:03:08 2014 +0000" }, "message": "Merge \"Remove use of pthread_cond_timedwait_monotonic.\"" }, { "commit": "4aeec176eaf11fe03f342aadcbb79142230270ed", "tree": "b2c7abd4b3624dc1f801f6a29893e0f66497f378", "parents": [ "a708e32a9f764a48175e705ec4bcd2201c84f492" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 27 16:09:46 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 28 11:35:02 2014 -0700" }, "message": "Refactor some GC code.\n\nReduced amount of code in mark sweep / semi space by moving\ncommon logic to garbage_collector.cc. Cleaned up mod union tables\nand deleted an unused implementation.\n\nChange-Id: I4bcc6ba41afd96d230cfbaf4d6636f37c52e37ea\n" }, { "commit": "afe4998fc15b8de093d6b282c9782d7182829e36", "tree": "bcb64fa7db43ece36a3e492deab5f9c162d39aa5", "parents": [ "67ad224199b9902b3e6d85737e03b679d1ab44f9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 27 10:55:04 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 27 15:24:44 2014 -0700" }, "message": "Change sticky GC ergonomics to use GC throughput.\n\nThe old sticky ergonomics used partial/full GC when the bytes until\nthe footprint limit was \u003c min free. This was suboptimal. The new\nsticky GC ergonomics do partial/full GC when the throughput\nof the current sticky GC iteration is \u003c\u003d mean throughput of the\npartial/full GC.\n\nTotal GC time on FormulaEvaluationActions.EvaluateAndApplyChanges.\nBefore: 26.4s\nAfter: 24.8s\nNo benchmark score change measured.\n\nBug: 8788501\n\nChange-Id: I90000305e93fd492a8ef5a06ec9620d830eaf90d\n" }, { "commit": "4d25df3f76f864b7629ac8c0046d46997f293d8d", "tree": "31840831d8c81d06ffd575fdb0adc8403cd8dbb1", "parents": [ "909f133bc938928a2403baccc983701cb9ebb17f" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Mar 21 17:44:46 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Mar 26 11:46:43 2014 +0100" }, "message": "Refactor deoptimization support in debugger\n\nThis CL prepares breakpoint support for inlined methods where we\u0027ll have to\ndeoptimize everything.\n\nWe move deoptimization-related information to Dbg class only (deoptimization\nrequest queue, full deoptimization event count and deoptimization lock). We\nreplace MethodInstrumentionRequest by DeoptimizationRequest. This is used to\nknow which kind of deoptimization is required for a particular event.\n\nIt also simplifies lock ordering a bit during event setup: we no longer need to\nhold the deoptimization lock while holding the breakpoint lock. Moreover, the\ndeoptimization lock should be held only after the event list lock.\n\nBug: 12187616\nChange-Id: Iff13f004adaeb25e5d609238bacce0b9720510e6\n" }, { "commit": "51b71028661092e8860cca4f8ca79848e03cdc2c", "tree": "ebb78fc3f6af61ba5d015ddaf0d28d6c3cbbbac1", "parents": [ "c5db77cce4138ac1f473872a06d230d876374337" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Mar 04 11:57:09 2014 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Mar 20 10:11:13 2014 +0000" }, "message": "Remove use of pthread_cond_timedwait_monotonic.\n\nUse posix compliant pthread_condattr_setclock instead.\nAlso, remove usage of HAVE_TIMEDWAIT_MONOTONIC and replace\nit with a specific reference to the only supported platform\nthat doesn\u0027t have it.\n\nChange-Id: I933f05c5b4965bab88ccd8e3e26c91549ed4184d\n" }, { "commit": "9758f79a6c1ef7f662caca9c1df39de1934166b8", "tree": "fdffb18605692c990665f3e99a17dcb0efb09893", "parents": [ "c1020433660737d466b0d726bbeb86d9a279a44a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 09:02:55 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 09:12:20 2014 -0700" }, "message": "Implement FINAL/OVERRIDE for clang.\n\nSeparate declaration from definition in certain places to work-around issues\nwith clang.\nRemove bogus lock annotation at definition in compilers.cc that is already\npresent at the declaration.\nRemove duplicate definition of ClassReference.\n\nChange-Id: I5368057bb36319a259110b2198610d9d2b2e5041\n" }, { "commit": "cfa5c4805b9e756aaa142047306bad2b13107834", "tree": "067839e9acc27de8c6c4abf7e5447ec943a50250", "parents": [ "63dcdff34c40a64df6939ea5bba71813abd2acd3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 12 14:11:40 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 12 14:20:31 2014 -0700" }, "message": "Fix tests for 64b\n\nThese tests make assumptions about pointer sizes.\n\nChange-Id: Idfd410144393d716ae176a6e9a51c5b414e0888e\n" }, { "commit": "4812d436ebf538043a7827253b2e940a52a43fcb", "tree": "18667db8756044a2625385b9ac3eab2a94451678", "parents": [ "b28bdd624b63a52e4a69b93fa9abee6fbd393610" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 11 12:42:25 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 11 12:43:22 2014 +0000" }, "message": "Fix BitVector::SetInitialBits().\n\nChange-Id: Iad3699d76ad69536aa4810329f5863bc995f6551\n" }, { "commit": "719d1a33f6569864f529e5a3fff59e7bca97aad0", "tree": "fcd84efd7b9806b93ec1a44e2317e6f882e7fe0e", "parents": [ "5365eea9940269b662cfbe103caa348816ff1558" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 12:13:39 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 06 19:16:01 2014 -0800" }, "message": "Enable annotalysis on clang ART builds.\n\nFix clang build errors aswell as restructure locking/mutex code for correct\nthread safety analysis support.\nReorder make dependencies so that host builds build first as they should\nprovide better compilation errors than target.\nRemove host\u0027s use of -fno-omit-frame-pointer as it has no value with correct\nuse of CFI, which we should have.\n\nChange-Id: I72cea8da9a3757b1a0b3acb4081feccb7c6cef90\n" }, { "commit": "6fac447555dc94a935b78198479cce645c837b89", "tree": "bcf1449999084b1e1dec3dac287f6f3670d7eda0", "parents": [ "7f0ff7e7fff82566bca5f9353eaa2c4f81f0671a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 25 17:01:10 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 26 16:38:22 2014 -0800" }, "message": "Make allocations report usable size.\n\nWork-in-progress to allow arrays to fill usable size. Bug: 13028925.\nUse C++11\u0027s override keyword on GCC \u003e\u003d 2.7 to ensure that we override GC and\nallocator methods.\nMove initial mirror::Class set up into a Functor so that all allocated objects\nhave non-zero sizes. Use this property to assert that all objects are never\nlarger than their usable size.\nOther bits of GC related clean-up, missing initialization, missing use of\nconst, hot methods in .cc files, \"unimplemented\" functions that fail at\nruntime in header files, reducing header file includes, move valgrind\u0027s space\ninto its own files, reduce number of array allocation routines.\n\nChange-Id: Id5760041a2d7f94dcaf17ec760f6095ec75dadaa\n" }, { "commit": "a1ce1fef2d49d1d537776a5308ace7102a815fe5", "tree": "a1afe850d9ebd21f4b751eb68e1ec3dc60dab001", "parents": [ "930f7b843ddc6e6530439d3fdb0e2133a6292f1e" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Mon Feb 24 23:23:58 2014 -0800" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Feb 26 13:17:44 2014 -0800" }, "message": "Split up CommonTest into CommonRuntimeTest and CommonCompilerTest\n\nChange-Id: I8dcf6b29a5aecd445f1a3ddb06386cf81dbc9c70\n" }, { "commit": "7cba217ab0661d74deccbb97160cdf60b74d4ea3", "tree": "509f912447ef4dcafc4f043267bf990cee62167c", "parents": [ "3ece181223974d8c081fb88cfbec8d85fd194d66", "ad0d30a2a2141aa0e9da9e97993ce20e4d8e056e" ], "author": { "name": "Bill Buzbee", "email": "buzbee@android.com", "time": "Mon Feb 10 18:18:04 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 10 18:18:04 2014 +0000" }, "message": "Merge \"Update to the BitVector Implementation\"" } ], "next": "ef7d42fca18c16fbaf103822ad16f23246e2905d" }