{"push_id":1763422955,"size":66,"distinct_size":66,"ref":"refs/heads/master","head":"3c72718b1a25b09a682e646ec035af1b39d279f4","before":"721cfe647f2997f61c67d105ddb5220347c8b6ba","commits":[{"sha":"4155fc3b9aa3d55d736b04063c042f300ca57bd9","author":{"name":"Andrew Kennedy","email":"8f4dbee1d1f8400df782dd784d2185d01f0bd28c@fb.com"},"message":"Fix generation of FPushClsMethodF bytecode, .adata processing\n\nSummary:\nFix generation of FPushClsMethodF bytecode. Reorganize the processing of `.adata` so that we have an explicit representation of the `.adata` declaration.\nThis involves introducing a pseudo-bytecode for all literal data.\n\nReviewed By: nbenton\n\nDifferential Revision: D5103747\n\nfbshipit-source-id: cfaf7f981747daa256c2a3ffddbb580f4fe6a636","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/4155fc3b9aa3d55d736b04063c042f300ca57bd9"},{"sha":"cbd64e4193b91df799569e7278d8312fcfc8631d","author":{"name":"Steve Walk","email":"39429c2911752f19c638fba99eacacf4adeb53fb@cavium.com"},"message":"Pass to reuse ldimmq immediate values\n\nSummary:\nThis adds a quick pass for ARM targets which attempts to reuse immediate values\ngenerated by the ldimmq Vinstr.  This pass only operates on single blocks.  The span\nof this optimization is tunable so that it does not increase register pressure.  The\ndefault for the new option is 8,  (-vEval.JitLdimmqSpan=8).\n\nHere is an example of the pass firing on test/zend/good/ext/intl/tests/grapheme.php\n\nWithout\n=====\n\n    36918     (38) DecRef<-> t7:Str\n    36919         Main:\n    36920               0x468005f4  d29ffe40              movz x0, #0xfff2\n    36921               0x468005f8  f2a05fe0              movk x0, #0x2ff, lsl #16\n    36922               0x468005fc  78e06b61              ldrsh w1, [x27, x0]\n    36923               0x46800600  11000421              add w1, w1, #0x1 (1)\n    36924               0x46800604  78206b61              strh w1, [x27, x0]\n    36925               0x46800608  d29ffe80              movz x0, #0xfff4         //<<--+\n    36926               0x4680060c  f2a05fe0              movk x0, #0x2ff, lsl #16 //<<--+\n    36927               0x46800610  78e06b61              ldrsh w1, [x27, x0]\n    36928               0x46800614  11000421              add w1, w1, #0x1 (1)\n    36929               0x46800618  78206b61              strh w1, [x27, x0]\n\nWith\n===\n\n    36929     (38) DecRef<-> t7:Str\n    36930         Main:\n    36931               0x200005f0  d29ffe40              movz x0, #0xfff2\n    36932               0x200005f4  f2a05fe0              movk x0, #0x2ff, lsl #16\n    36933               0x200005f8  78e06b61              ldrsh w1, [x27, x0]\n    36934               0x200005fc  11000421              add w1, w1, #0x1 (1)\n    36935               0x20000600  78206b61              strh w1, [x27, x0]\n    36936               0x20000604  91000800              add x0, x0, #0x2 (2)  //<<---\n    36937               0x20000608  78e06b61              ldrsh w1, [x27, x0]\n    36938               0x2000060c  11000421              add w1, w1, #0x1 (1)\n    36939               0x20000610  78206b61              strh w1, [x27, x0]\n\nThis was tested by running the regression tests with 6 different option sets.\nNo regressions were introduced.\n\nPerformance testing is pending.\nCloses https://github.com/facebook/hhvm/pull/7637\n\nDifferential Revision: D4501447\n\nPulled By: mxw\n\nfbshipit-source-id: 4f08109ddef35da85209b1ae09f40a4181353513","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/cbd64e4193b91df799569e7278d8312fcfc8631d"},{"sha":"1b2f66fc396851c4e58480981f8ba41625aa89d8","author":{"name":"Max Wang","email":"7c2db0bd44ce1e75d861a88bee8427961fa5e3cd@fb.com"},"message":"More API cleanup in ArrayInit, ArrayData, Array\n\nSummary:\nThis diff makes three separate changes which are transitively closed\nover interdependency:\n- Add TypedValue overloads of *ArrayInit::setValidKey().\n- Take `Cell key, Cell value` over `const Variant& key, Cell value` for\n  the TypedValue overloads of ArrayData::{set,add}().\n- Add TypedValue overloads of ArrayData::Create().\n\nReviewed By: jano\n\nDifferential Revision: D5106153\n\nfbshipit-source-id: 2acf791318f5b5e63651600c17058a399749cb5f","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/1b2f66fc396851c4e58480981f8ba41625aa89d8"},{"sha":"5fc23e702fe70982bbead4a047d0c3582d944af5","author":{"name":"Max Wang","email":"7c2db0bd44ce1e75d861a88bee8427961fa5e3cd@fb.com"},"message":"Use TypedValue instead of const Variant& in appendWithRef()\n\nReviewed By: jano\n\nDifferential Revision: D5109543\n\nfbshipit-source-id: f494dc84ec96e31b282c13cad6a0a6faa8824271","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/5fc23e702fe70982bbead4a047d0c3582d944af5"},{"sha":"b1ad77de24b5ae32682aa347ca95af2451974bd2","author":{"name":"Max Wang","email":"7c2db0bd44ce1e75d861a88bee8427961fa5e3cd@fb.com"},"message":"Cleanup for {is,get}{Int,Str}Key() in array-data-defs.h\n\nSummary: Follow-up to an earlier diff.\n\nReviewed By: jano\n\nDifferential Revision: D5109729\n\nfbshipit-source-id: 37366e986550a8397734cc59a55cabf797442269","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/b1ad77de24b5ae32682aa347ca95af2451974bd2"},{"sha":"8245a2d9ff595b9576975745ce7cef1ed8d0e463","author":{"name":"Jim Saxman","email":"c9cf567232d8ed37426e25df9cd58e4e9e5289c0@qti.qualcomm.com"},"message":"Implement StringData::hashHelper in AArch64 assembly\n\nSummary:\nStringData::hashHelper() is implmented in x64 assembly and used\nwhen NO_M_DATA is set, which in turn is controlled by USE_LOWPTR.\nThis patch provides an equivalent version for Aarch64.\n\nThere are no new errors in the unit tests, which were tested with\n{-m interp, -m pgo, -r, and empty} and OSS Performance suite still\nruns fine, both with and without USE_LOWPTR and ENABLE_AARCH64_CRC\ndefined.\nCloses https://github.com/facebook/hhvm/pull/7809\n\nDifferential Revision: D5013451\n\nPulled By: mxw\n\nfbshipit-source-id: ab6e3f5ba7ced9a909f6935a0eb63265a1b38c41","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/8245a2d9ff595b9576975745ce7cef1ed8d0e463"},{"sha":"2a01781a26ca1f80c6292f22bf5aca7f7f1edfc3","author":{"name":"Bill Nell","email":"3aa3615152ac5bed4c0a2ada000f815616ef48f3@fb.com"},"message":"Remove bogus TODO comment from debugger code.\n\nSummary: Remove bogus TODO comment from debugger code.\n\nReviewed By: yinghuitan\n\nDifferential Revision: D5106418\n\nfbshipit-source-id: bc358085a60fb253ad3a23b4ed6020b43983cb59","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/2a01781a26ca1f80c6292f22bf5aca7f7f1edfc3"},{"sha":"afac43b2a2540a49c7e924f52cdd66577a7a659a","author":{"name":"Michael O'Farrell","email":"07ba10442024df486e79fbdd20c25af15256968b@fb.com"},"message":"Factor hashtable implementation out of MixedArray and SetArray.\n\nSummary: Moves hashtable implementation to one common location.\n\nReviewed By: alexeyt\n\nDifferential Revision: D5111842\n\nfbshipit-source-id: 095795a2a66638a48335c105de5a1a71197af836","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/afac43b2a2540a49c7e924f52cdd66577a7a659a"},{"sha":"1af605ef001397d3084951baa4220c298853a232","author":{"name":"Oguz Ulgen","email":"388d215575b4c419b16fa283c1b704722a0db802@fb.com"},"message":"Implement codegen for references\n\nSummary: Implement codegen for references\n\nReviewed By: vladima\n\nDifferential Revision: D5099018\n\nfbshipit-source-id: 8686c7e1d6191fb51a54484a42249e337470ca57","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/1af605ef001397d3084951baa4220c298853a232"},{"sha":"47e882aa0b62630fbc2602bd31296719df2ed94a","author":{"name":"Michael O'Farrell","email":"07ba10442024df486e79fbdd20c25af15256968b@fb.com"},"message":"Up php version number from 7.0 to 7.1\n\nSummary: We want to target the latest PHP7 not the past one.\n\nReviewed By: paulbiss\n\nDifferential Revision: D5103767\n\nfbshipit-source-id: 6b82b7107742bc1477d1875d058f2f640658be03","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/47e882aa0b62630fbc2602bd31296719df2ed94a"},{"sha":"fe1f63ea3f7a0ddfeaa02f1e8372bf0e09f042d7","author":{"name":"Xiaochen Pan","email":"7bf9621b295e22100107979b58ba47328e141dbf@fb.com"},"message":"add curl share interface implementation\n\nSummary: add curl share interface implementation\n\nReviewed By: paulbiss\n\nDifferential Revision: D4942241\n\nfbshipit-source-id: e76b762fa41665aa77c8e84eaadeb4248892950a","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/fe1f63ea3f7a0ddfeaa02f1e8372bf0e09f042d7"},{"sha":"17ac154dd41cb8047fd073311e056a459377454a","author":{"name":"Edwin Smith","email":"c8769332ce828c2bdb33494642fa5a79013d103e@fb.com"},"message":"Remove unnecessary Array::exists() and remove() overloads\n\nSummary:\nUsing = delete here generates an ambiguity error when callsites\nare passing int; just delete them completely to match other\nmethods. Callsites using char, short, or bool should still\nimplicitly cast to int like before.\n\nReviewed By: alexeyt\n\nDifferential Revision: D5112233\n\nfbshipit-source-id: abdd5ee125ad6243be41a01d055e5fc3356df9e1","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/17ac154dd41cb8047fd073311e056a459377454a"},{"sha":"38a77cf9e00b9699ad96e2a9705ec554e6f1ddd5","author":{"name":"Jacob Bower","email":"ec7ba04b530f08d3f3f0b78b23b46b37d6010ce7@fb.com"},"message":"Fix incorrect erasure from s_activeSubscriptions\n\nSummary: As markw65 pointed out in D5091834, this should have been updated when s_activeSubscriptions was turned from a vector into an unordered_map.\n\nReviewed By: markw65\n\nDifferential Revision: D5112492\n\nfbshipit-source-id: 6addc784ceee2ff48037c22504e3692be7fea140","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/38a77cf9e00b9699ad96e2a9705ec554e6f1ddd5"},{"sha":"5155049d79b6b5525a08fa1013e05c0c3cf66f87","author":{"name":"Alexander Chow","email":"00259f7a596adebf55502551b755a253cdaf7719@fb.com"},"message":"server really logs hostname\n\nSummary: server really logs hostname\n\nReviewed By: dlreeves\n\nDifferential Revision: D5082768\n\nfbshipit-source-id: df86b243f289b3d6dd979c5114a47f245f129f2b","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/5155049d79b6b5525a08fa1013e05c0c3cf66f87"},{"sha":"d2dbbc91908d75a7448cbfd585cd1b52757bf247","author":{"name":"Alexander Chow","email":"00259f7a596adebf55502551b755a253cdaf7719@fb.com"},"message":"Log client out of date to monitor logs\n\nSummary: Log client out of date to monitor logs\n\nReviewed By: arxanas\n\nDifferential Revision: D5097408\n\nfbshipit-source-id: 5e81638dd6ec0196c6ca2aeefc8abed22e0aedc0","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/d2dbbc91908d75a7448cbfd585cd1b52757bf247"},{"sha":"d5427b1e7cf6d1f45c11bac5ec50938c7a05eaa8","author":{"name":"Daniel Andersson","email":"b8d352f3a3ed6ef0b50e565a4f0337ac47c3fb50@fb.com"},"message":"add walltime to perf logging\n\nSummary: Add wall-time, analogous to cpu-time counters in ODS and StructLog.\n\nReviewed By: intellectuallyswole\n\nDifferential Revision: D5113083\n\nfbshipit-source-id: 020a3222a7d11ab041210ad1b5ac4dc8f47e8cb3","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/d5427b1e7cf6d1f45c11bac5ec50938c7a05eaa8"},{"sha":"cd6d9a34b2ccac8213bbd5652e0d8298eb676a24","author":{"name":"Christoph Muellner","email":"ed9ac63a69f2c38cce6ce9203649295e89e79d0a@theobroma-systems.com"},"message":"jit: arm64: Fix regression in async testcases (repo auth)\n\nSummary:\nA recent PR (#7833/61b3b0d) addresses the problem, that the arm64\nlowering/emitter code need to truncate the operands of the vasm\ninstructions cmpb{} and cmpbi{}. The actual problem is, that the\ngiven values are not Vreg8 values, but treated as Vreg8 values\nby follow-up code. The fix in the named PR is to insert a movtqb{}\nwhenever a type value is lowered.\n\nAs reported by swalk-cavium this breaks some async tests when\nrunning in repo authoritative mode.\n\nThis patch addresses this issue by clearing bits 9-31 of the\ntype value, but keep bits 32-63 to keep the AUX value in the\nregister (clearing that bits leads to the regression in repo\nauthoritative mode).\n\nThe patch relies on the fact that the type value will later\nbe interpreted as 32-bit (W) register, i.e. ignoring the AUX\nvalue anyways. Therefore a cmpb{} and cmpbi{} will not see\nthe AUX value.\n\nRunning the 'all' test set showed no regressions with and without\nauthoritative mode (all previously failing async tests pass now).\n\nThanks mxw for the idea for this PR.\nCloses https://github.com/facebook/hhvm/pull/7851\n\nDifferential Revision: D5112044\n\nPulled By: mxw\n\nfbshipit-source-id: d573565702490d78292bda587e8be48ed4365e21","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/cd6d9a34b2ccac8213bbd5652e0d8298eb676a24"},{"sha":"14f3129d0a16ef9633f5ee9f788a511c99cbee5a","author":{"name":"Jake Bailey","email":"f0e59981ca794ff721ad24cc2eb30ca47d3b6f04@fb.com"},"message":"Allow destructuring of varrays\n\nSummary: Allow varrays as the rvalue in list expression (destructuring) assignments.\n\nReviewed By: kmeht\n\nDifferential Revision: D5114923\n\nfbshipit-source-id: c0f36097462d487efbe140dce9091a84d6e2cf31","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/14f3129d0a16ef9633f5ee9f788a511c99cbee5a"},{"sha":"3880b958237d0eaa259c6b6da3c7db6b33d0babf","author":{"name":"Brett Simmers","email":"028739e96ccaf586e6f341efb813e8f7cc738ded@fb.com"},"message":"Mark socket_connect_params as flaky\n\nSummary: It tries to listen on a random port without any retry logic.\n\nReviewed By: kodafb\n\nDifferential Revision: D5105308\n\nfbshipit-source-id: ae1c8e970b27010ec7b4385bddf12908d2dfc599","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/3880b958237d0eaa259c6b6da3c7db6b33d0babf"},{"sha":"8918e1db432c884ebab491f9a96226acfe68fb8c","author":{"name":"Steve Walk","email":"39429c2911752f19c638fba99eacacf4adeb53fb@cavium.com"},"message":"Aarch64 peephole using csinc\n\nSummary:\nThis change introduces an Aarch64 specific pseudo instruction which\nis a variation of the conditional move.  One pattern seen in the OOPSLA'14\nbenchmark tests involved the expression  like this in C\n\n    (cond) ? exp : 1;\n\nThe hard-coded zero register can be used with the csinc instruction to\ncreate the 1.\n\nBefore\n======\n     0x146010d0  39c0e361             ldrsb w1, [x27, #56]\n     0x146010d4  72001c3f             tst w1, #0xff\n     0x146010d8  52800022             movz w2, #0x1   //<<---\n     0x146010dc  1a810041             csel w1, w2, w1, eq\n\nAfter\n=====\n      0x362011d0  38e16b61              ldrsb w1, [x27, x1]\n      0x362011d4  72001c3f              tst w1, #0xff\n      0x362011d8  1a9f1421              csinc w1, w1, wzr, ne\n\nThis saves 1 instruction per instance.\n\nThe standard regression suite was run with 6 option sets.  No new regressions\nwere introduced.\nCloses https://github.com/facebook/hhvm/pull/7823\n\nDifferential Revision: D5112346\n\nPulled By: mxw\n\nfbshipit-source-id: 44cebfb34868523c1bda9615cd4605beb34f06ed","distinct":true,"url":"https://api.github.com/repos/shantanusharma/hhvm/commits/8918e1db432c884ebab491f9a96226acfe68fb8c"}]}
{"actor":{"display_login":"shantanusharma"}}