move declaration of cpu_set_t to ext_hotprofiler.h to fix OS X build

Summary: Fixes issue https://github.com/facebook/hhvm/issues/3079

cpu_set_t is used in ext_hotprofiler.h (line 18 of .cpp, line 129 of ext_hotprofiler.h) before it has been defined in ext_hotprofiler.cpp (at line 51).
Closes #3106

Reviewed By: @ptarjan

Differential Revision: D1422472

Pulled By: @fredemmott
flow stop, servermsg out of stop, flow command modules

Summary: Move nice_kill from ClientStop command functor to param module.

Reviewed By: @jwatzman

Differential Revision: D1421357
Make hhvm print errors and abort if an HNI systemlib is broken

Summary: A parse error currently just leads to everything in the systemlib not
being defined.

Reviewed By: @ptarjan

Differential Revision: D1421895
Fix support for parse_str; treat it like extract

Summary: Assume it trashes the variable environment, and in repo mode make it
part of the DisallowDynamicVarEnvFuncs scheme.  Also close a few holes in this
stuff in fb_rename_function and simple_function_call.cpp.

Closes #3081.

Reviewed By: eletuchy, ptarjan

Differential Revision: D1422197
Remove uses of RegNumber outside X64Assembler.

Summary: Prefer Reg64 (etc) instead.

Reviewed By: @oyamauchi

Differential Revision: D1421690
Remove blocking acquire of the write-lease in smashPrologueGuards

Summary: It was only needed to unprotect the tc, which we no longer do.

Reviewed By: @swtaarrs

Differential Revision: D1424510
Move ifThenElse helpers to code-gen-x64.cpp

Reviewed By: @jasone

Differential Revision: D1424833
Implement zend EG(error_reporting).

Summary: Use a macro and wrapper class to turn lval/rval use of EG(error_reporting)
into calls to setter/getter methods for class RequestInjectionData.
Closes #3126

Reviewed By: @swtaarrs

Differential Revision: D1424013

Pulled By: @fredemmott
Extend zend_call_function to handle method invocation.

Summary: Add a null pointer guard in the zend property getter,
as for example when the object doesn't have the given property.
Closes #3076

Reviewed By: @ptarjan

Differential Revision: D1413661

Pulled By: @LiraNuna
Introduce c_WaitHandle::asWHKind()

Summary: Move all static casts from c_WaitHandle to c_FooWaitHandle to a single
function called c_WaitHandle::asFoo(). Makes all these huge switches
more readable.

Reviewed By: @billf

Differential Revision: D1425048
Clear the locals vector in FrameState when translating a block with an unprocessed predecessor

Summary: We were clearing the snapshot in FrameState, but not FrameState::m_locals.

Reviewed By: @ottoni

Differential Revision: D1409019
Cleanup mcg creation/deletion

Summary: We recently added a call to mcg->code.unprotect(), with no
explanation of why it was needed, leading to a long debate. This moves
it to a more appropriate place, and adds a comment to explain why
its there.

We also weren't ever deleting JIT::mcg, even though its destructor does
real work (such as deleting the perf-pid.map file, if requested), and also
freeing memory (which will help with leak detection).

Reviewed By: @swtaarrs

Differential Revision: D1424033
no tramps, always far-call

Summary: Removing trampolines completely; rip-relative indirect calls seem better and simpler.

Reviewed By: @markw65

Differential Revision: D1352053
typechecker: target line-clear term sequences to ttys only

Summary: * ensure that the line-clearing command is not invoked with any ##--from##
 * line-clearing ansi sequence is private to the module; \n should be used for non-tty
 * renamed ttyColor.ml to tty.ml, since it houses other functionality
 * moved tty-related code from utils.ml to tty.ml

Reviewed By: @pikatchu

Differential Revision: D1426336
Raise warning when calling get_called_class outside of a class

Summary: PHP5 raises a warning when `get_called_class` is called without a class.

Reviewed By: @ptarjan

Differential Revision: D1423243
reword CLA sentence in README.md

Summary: slight change to CLA sentence so ptarjan can test his external reviews hook
Closes #3137
Created from: https://github.com/facebook/hhvm/pull/3137

Reviewed By: @ptarjan

Differential Revision: D1423793
Fix for Redis EVAL / EVALSHA

Summary: phpredis casts nil responses to bool(false), whereas HHVM returns NULL for both
false and nil return values. For example:

    <?php
    $redis = new Redis();
    $redis->connect( '127.0.0.1' );
    var_dump( $redis->evaluate( 'return' ) );
    var_dump( $redis->evaluate( 'return false' ) );

With Zend / phpredis, the output is:

    bool(false)
    bool(false)

With HHVM, it's:

    NULL
    NULL

This patch adds a fix and a test.
Closes #3127

Reviewed By: ptarjan, bsimmers

Differential Revision: D1422528

Pulled By: @fredemmott
De-templatize testimm since IndexedMemoryRef is gone.

Reviewed By: @bertmaher

Differential Revision: D1426437
Replace bogus uses of AttrFinal with AttrNoOverride

Summary: We want to guard on objects that can't be overridden, not if they're
marked final.

Reviewed By: jdelong, mwilliams

Differential Revision: D1425605