GitHub PushEvent at 2014-06-05 15:32:18
Repository: hhvm
Actor: oprudkyi
Payload: {"shas":[["fd943295cf6c278e5fbd78c282fa515621f791c3","1516b4a1916f34160b7ec7191ddd2c2f69a2298f@fb.com","Fix pcre cache to not fatal when full\n\nSummary: So pcre cache can only hold 96x1024 items, when we hit full we raise an\nerror. From what I can see there isn't a good reason we can't just\ncontinue.\n\nIf one is doing 98,304 unique regular expressions you probably don't\ncare about regular expression caching.\n\nSince we're using AtomicHashArray we can't purge anything without adding\na lock and we'd probably want to do LRU rather than what PHP does which\nis FIFO eviction.\n\nAlso changed the code to check Folly's full parameter rather than\nrelying on calling .size().\n\nReviewed By: @ptarjan\n\nDifferential Revision: D1168192","Scott MacVicar",true],["52cb972b7436f33b7321bd1f1a33951c0ead9573","7647b3f0de2c5f049f3fd8683da596016764b513@fb.com","Optimize checks in iter_next_packed also\n\nSummary: I realized I could play the same trick as D1340787 for\niter_next_packed.\n\nReviewed By: aravind\n\nDifferential Revision: D1343589","Bert Maher",true],["cbd8f584b8b185594b15523d49a4030c12cecc6c","4baaee8a3e1d52ead025808fe14e6a867853d49e@fb.com","Create a CodeGenFixups class\n\nSummary: Create a CodeGenFixups class, to hold temporary information that will eventually be put in global tables. Put m_pendingFixups, m_pendingCatchTraces and a newly created m_pendingJmpTransIDs there.\n\nThe goal is to have a single place to hold information that may need to be fixed up if we modify the generated code before committing it (eg by relocating it, or by peephole optimizations). Also, these are things that must not go into the global tables if we end up rolling back.\n\nReviewed By: @ottoni\n\nDifferential Revision: D1347532","mwilliams",true],["a4111626ba57e69e53964cfcc3fd87d66bd8ab3a","4baaee8a3e1d52ead025808fe14e6a867853d49e@fb.com","Fix clang build\n\nSummary: thats it\n\nReviewed By: @dariorussi\n\nDifferential Revision: D1348180","mwilliams",true],["d336fb891f03a5b3795031169b18f47892a9ad41","4baaee8a3e1d52ead025808fe14e6a867853d49e@fb.com","Get rid of SmashTarget\n\nSummary: The SmashTarget is a short lived, malloced data block that gets passed\nto handlePrimeCacheInit, telling it how to smash the call. The information it\ncontains is the address of the movq instruction that needs smashing, and the\nreturn address of the handlePrimeCacheInit method.\n\nWe can find the return address by inspecting the stack, and then we can\npass in the offset from that to the movq instead of passing in the address\nof a SmashTarget.\n\nIn addition to avoiding the malloced data, this has the nice effect of making\nall the code position independent too.\n\nReviewed By: @mxw\n\nDifferential Revision: D1348201","mwilliams",true],["71ffbb7b0ffd0229f43eb90803368687860927be","4baaee8a3e1d52ead025808fe14e6a867853d49e@fb.com","Fix crash on missing trait in RepoAuthoritative mode\n\nSummary: At some point we stopped checking up front that\ntraits exist in RepoAuthoritative mode. But we still assume\nthat Unit::lookupClass will find them, and crash if not.\n\nThis checks every trait, and fatals if it doesn't exist.\n\nReviewed By: @dariorussi\n\nDifferential Revision: D1348353","mwilliams",true],["957dd9ea65dc3c6c7e5a0ae749320030019db55f","260b10ddf1bd1f500597a590597cdf54c83bffb6@fb.com","Fix for Xenon crash, change order of decRef between locals and $this\n\nSummary: Making lifetime of $this longer so that a stack walk from Xenon can still find the class and a somewhat usable $this\n\nReviewed By: jan, mwilliams\n\nDifferential Revision: D1343247","Dario Russi",true],["430a584e13d324e0af3d0a944a930a025bdb53b6","f008e9e5081849eca34e692e38a6c8a66fd79d1f@bauerj.eu","added 'Z' format string for pack/unpack\n\nSummary: Since PHP 5.5 there is a 'Z' format string for (un)pack with equivalent functionality to 'a'.\n\nHHVM has not implemented this yet.\n\nSee: http://docs.hhvm.com/manual/en/function.pack.php\nCloses #2713\n\nReviewed By: @ptarjan\n\nDifferential Revision: D1345185\n\nPulled By: @JoelMarcey","Johann Bauer",true],["ef5fab154fe604160a7280eca1f32c468f5ee8df","1459d19acca8aa0d331ced91d7cdfdd61263a03a@fb.com","Add test for PHP subclass type strictness behavior\n\nSummary: lolwut\n\nReviewed By: @ptarjan\n\nDifferential Revision: D1347668","Fred Emmott",true],["61907f4a55bd8e93834ddb41ddd201356f2e6b8b","088e16a1019277b15d58faf0541e11910eb756f6@simon.geek.nz","Ignore /bin\n\nSummary: Folder is generated by cotire\nCloses #2755\n\nReviewed By: @fredemmott\n\nDifferential Revision: D1348999\n\nPulled By: @JoelMarcey","Simon Welsh",true],["728e485af2a1dc8311924ed1247585902668c90e","608a0426cfdb46c29ac39909e0d1ac78f6a14462@fb.com","Truncate regions before type assert opcodes\n\nSummary: analyze() was doing this but the tracelet selector wasn't. Luckily\nit's a super simple fix. This was causing us to miss out on some type\ninformation from hhbbc.\n\nReviewed By: ottoni, jdelong\n\nDifferential Revision: D1347717","bsimmers",true],["9d1b742bea1e27d589cf2ae5dc7131f83e5584d0","2e25df8f749a1c4ba79a00a563a2df531f3e63b2@fb.com","Convert null_object/resource references\n\nSummary: We've been using null_* when we really want to just return an\nempty/new Object/Resource/String/Array. This is in preparation\nfor a codemod that will convert these sites to null_foo_value but gets\nthe short-term win. Handle Variant in a separate diff because it of the\nsemantics.\n\nReviewed By: @markw65\n\nDifferential Revision: D1344569","Sean Cannella",true],["7485839688bbe0678a542b1e9dab64fd48bf2f1e","b5f297ae1db97129134ea299877e232142939f36@debian.org","CMake: install headers under DESTDIR\n\nSummary: The relatively new HHVM_INSTALL_HEADERS function has some special code\nto recursively copy files to the target, using file(COPY ). However,\nfile(COPY) does not by default prefix with DESTDIR and would install the\nheaders into the system paths, instead of under the supplied destination\ndirectory.\n\nChange the function to prefix the target with $ENV{DESTDIR} to fix this.\n\nThis matters especially for downstream packagers, as packages usually\nbuild by running \"make install\" using a DESTDIR, then packaging that\ndirectory up. Without this, compilation would fail, as the build system\nwould attempt to install in a privileged system directory.\nCloses #2777\n\nReviewed By: @fredemmott\n\nDifferential Revision: D1349240\n\nPulled By: @JoelMarcey","Faidon Liambotis",true],["3fc46fff0edf3fd7c6455a411a05d5ac1b7ded13","b5f297ae1db97129134ea299877e232142939f36@debian.org","Remove CMake/FindLibEvent.cmake.bak\n\nSummary: CMake/FindLibEvent.cmake.bak is presumably a backup file from an older\nversion of CMake/FindLibEvent.cmake that has been commited by accident\nduring the initial open-source release (commit c3f1ab2).\n\nRemove it as it's obviously redundant.\nCloses #2776\n\nReviewed By: @fredemmott\n\nDifferential Revision: D1349226\n\nPulled By: @JoelMarcey","Faidon Liambotis",true],["f9898fe5b9ce22428fdb1bbd6d4708bbf189ed06","2e25df8f749a1c4ba79a00a563a2df531f3e63b2@fb.com","Convert Variant references\n\nSummary: Use init_null() when returning Variant, and uninit_null() when\nreturning Variant() to make intent more clear and save some work.\n\nReviewed By: @markw65\n\nDifferential Revision: D1344927","Sean Cannella",true],["82adfc624ec56b5bce998700b6cc86743af9bafc","14e793d896ddc8ca6911747228e86464cf420065@fb.com","Kill AsyncFunctionWaitHandle::{get,set}PrivData()\n\nSummary: Kill unused m_privData in AsyncFunctionWaitHandle. Does not save any memory\nbecause of 16-byte alignment.\n\nReviewed By: @billf\n\nDifferential Revision: D1344989","Jan Oravec",true],["b307e8f2beed97eba0b890a18d222e39f90e8332","608a0426cfdb46c29ac39909e0d1ac78f6a14462@fb.com","Fix test/quick/redeclared_function.php.expectf\n\nSummary: The path was too specific\n\nReviewed By: aravind\n\nDifferential Revision: D1349678","bsimmers",true],["0c4d89ce21536e88eebd91dc711e1dc700e0fdd3","1650efe4630bf7b93325107599da3e871b6859a7@fb.com","cut session_id() size by 1/2\n\nSummary: We were converting it to ascii hex first, then converting that to the final encoding they ask for. Lets stay in binary until the final encoding.\n\nCloses #1884\n\nReviewed By: @JoelMarcey\n\nDifferential Revision: D1347195","Paul Tarjan",true],["9b6e9514c57c9acd4daa10b82a09d8f6bfd60830","1650efe4630bf7b93325107599da3e871b6859a7@fb.com","don't kill shutdown functions when you want a session one\n\nSummary: Closes #2725\n\nReviewed By: @fredemmott\n\nDifferential Revision: D1347393","Paul Tarjan",true],["5460439089e42361cfd5160c45f0fb4c6956e625","7c7aaadbbffd562b7b08991adb058767a9461b6a@fb.com","Some eta conversion cleanup\n\nSummary: Saw it, didn't like it.\n\nReviewed By: @swtaarrs\n\nDifferential Revision: D1349390","Alex Malyshev",true]],"size":185,"ref":"refs/heads/master","head":"20e4b5564803eb987bd952097c5eda214f1dd28a"}