Fixed Exception in DoublyLinkedList::offsetUnset

Summary: zendtests are hopefully enough test coverage
Closes https://github.com/facebook/hhvm/pull/3391

Reviewed By: @fredemmott

Differential Revision: D1484805

Pulled By: svcscm
Fix an argument ordering bug in Redis lrem.

Summary: The PHP interface takes arguments in a different order than the redis server expects them.
Interchange (0-based) args 1 and 2 keyed off of the command name.

This fixes https://github.com/facebook/hhvm/issues/3357

A scan through the Redis PHP API documentation leads me to believe this is the only
instance wherein the PHP arguments are in a different order than the server arguments.
Closes https://github.com/facebook/hhvm/pull/3358

Reviewed By: @LiraNuna

Differential Revision: D1476999

Pulled By: svcscm
Use standard http reason for replay server

Summary: Closes https://github.com/facebook/hhvm/pull/3387

Reviewed By: @ptarjan

Differential Revision: D1484823

Pulled By: svcscm
Fix #3053: rename toString in ReflectionFunction / ReflectionMethod

Summary: This is a continuation of #3312 with tests.
Closes https://github.com/facebook/hhvm/pull/3356

Reviewed By: @LiraNuna

Differential Revision: D1476373

Pulled By: svcscm
Changed definitions of DateTime constants to null string (from 0).

Summary: Fixes issue #3439
Did not find any hhi specific tests, so no test changes were made.
Closes https://github.com/facebook/hhvm/pull/3442

Reviewed By: @fredemmott, @jwatzman

Differential Revision: D1492119

Pulled By: svcscm
Updated the twital framework tests to 0.1.6

Summary: Closes https://github.com/facebook/hhvm/pull/3450

Reviewed By: @fredemmott

Differential Revision: D1494805

Pulled By: svcscm
fix tc-space accounting

Summary: Fix for double counting for the size of some parts of tc when computing total size. The total size number for TC-space was double counting dat,rds,and peristentRDS.

override-test-failures

Reviewed By: @bertmaher

Differential Revision: D1480884
Implement SplDoublyLinkedList::add

Summary: Closes https://github.com/facebook/hhvm/pull/3390

Reviewed By: @LiraNuna

Differential Revision: D1484817

Pulled By: svcscm
Expect object or string in class_implements

Summary: Closes https://github.com/facebook/hhvm/pull/3393

Reviewed By: @fredemmott

Differential Revision: D1484793

Pulled By: svcscm
Set Cwd to the realpath, exclude all "../" etc.

Summary: Closes https://github.com/facebook/hhvm/pull/3424

Reviewed By: @fredemmott

Differential Revision: D1488966

Pulled By: svcscm
GMP cleanup/fixes

Summary: - Fixes a memory leak in gmp_init
- Removes uses of mpz_clears/mpz_inits, which some GMP versions do not have (should fix #3421)
- C++ functions that should be static are now static
- Re-adds miss-merged lint fixes
Closes https://github.com/facebook/hhvm/pull/3425

Reviewed By: @fredemmott

Differential Revision: D1488974

Pulled By: svcscm
Importer now downloads a copy of php release zip for consistant import

Summary: So all developers can sync between the tests being imported, the importer now downloads the correct version of the php release.

Reviewed By: @ptarjan

Differential Revision: D1494054
Corrected a tokenizer test

Summary: thatwaseasy

Reviewed By: @ptarjan

Differential Revision: D1494168
Corrected the coercion mode of token_get_all

Summary: thatwaseasy

Reviewed By: @ptarjan

Differential Revision: D1494182
Determine AttrHot for profiling translations in a better way

Summary: Right now, during interpreter-based type-profiling, we determine whether
to mark a function with AttrHot based on whether we enter that function
more than EvalHotFuncThreshold (40) times.

This is bad because it is a function of the mix and number of requests
we run during profiling phase, and having more functions marked as
AttrHot can cause regressions.

This diff changes it so that AttrHot is set by computing the
"hottest" EvalHotFuncCount functions, where hotness is
determined by the profile count of the function after warmup.

Reviewed By: @edwinsmith

Differential Revision: D1484134