Check for post test warnings too.

It looks like we fixed the post test fatal problem that joomla was having, but now yii may have post test warnings. For example,

/data/users/ptarjan/fbcode/hphp/test/frameworks/frameworks/yii/tests/./
framework/web/auth/CWebUserTest.php
HipHop Warning: Constant PHPUNIT_COMPOSER_INSTALL already defined in ve
ndor/phpunit/phpunit/composer/bin/phpunit on line 49

Let's see if this can fix that.

Reviewed By: @alexmalyshev

Differential Revision: D1098415
Refactor func.h

Reviewed By: @paroski

Differential Revision: D1075910
Sigh. More post test warning fun for yii.

I love yii. I love yii. I love yii.

Need to check for another type of post test warning, that includes a PHPUnit Exception.

Reviewed By: @ptarjan

Differential Revision: D1098761
Make the user run command output better

The way I was generating the "RUN TEST FILE" commands at the end of the script run or within .errors/.fatals files was not the best. This improves that.

Reviewed By: @ptarjan

Differential Revision: D1098631
Make PhysReg able to represent 64 registers

Finally. PhysReg can now represent 32 GP regs and 32 SIMD regs.

Having the arch() dependency inside PhysReg feels a little strange, but
(a) Map really does need to behave differently depending on the
architecture, and (b) it's really convenient to have Map defined inside
PhysReg. I think it's an okay tradeoff.

In a followup diff, I'll get rid of x2a and have PhysRegs with ARM
register numbers flowing through XLS. It's still going to be a bit messy
because both codegen backends use the operator[] trick to create memory
references from register + offset. The ARM backend uses
vixl::Register::operator[], but the x64 backend uses
PhysReg::operator[]. I'm not yet sure how I'll reconcile this.

Reviewed By: @ottoni

Differential Revision: D1097959
Pass inGenerator flag in RetCtrl and FunctionExitSurpriseHook

Code gen for FunctionExitSurpriseHook and refcount optimization RetCtrl
opcode processing needs to know whether these opcodes were used inside a
generator. They currently obtain this information thru curFunc(). This
information is known statically, so pass it explicitly.

Needed for unified inner/outer generator functions.

Reviewed By: @jdelong

Differential Revision: D1095196
chmod -x runtime/vm/jit/*.{cpp,h}

I don't know why this keeps happening.

Reviewed By: @jdelong

Differential Revision: D1098912
Disable flaky zend test

Reviewed By: @jdelong

Differential Revision: D1099379
Fix a bug in return type inference: ctors can be inherited from abstract classes

I think I was thinking that since interfaces can't declare
__construct methods, abstract classes probably couldn't either.
Actually not true.

Reviewed By: @dariorussi

Differential Revision: D1092876
Make IterRange work with range-based for

Give it a begin() and end() function.

Reviewed By: @edwinsmith

Differential Revision: D1093982
Minor const correctness fix in hhbc.{h,cpp}

Reviewed By: @alexmalyshev

Differential Revision: D1094436
Improve c_Map::reserve

This diff tightens up the logic in c_Map::reserve to be more careful when
mixing signed and unsigned integers and when mixing 32-bit and 64-bit
integers.

This diff also removes a bogus assert, adds some more (non-bogus) asserts,
and adds some comments.

Reviewed By: anietoro

Differential Revision: D1097684
Fix how the bytecode emitter handles foreach loops inside fault funclets

The bytecode emitter had a bug where it wasn't properly keeping track of
which iterators were live at a fault funclet's entry point. When emitting
a foreach loop inside the body of a fault funclet, the bytecode emitter in
some cases would erroneously use an iterator variable ID that was already
live.

This diff fixes the bug by updating the bytecode emitter to keep track of
which iterators were live at a fault funclet's entry point, and by updating
the logic for allocating iterator variable IDs to correctly handle foreach
loops inside fault funclets.

Reviewed By: @jdelong

Differential Revision: D1099508
Fold HPHP::Transl's contents into HPHP::JIT

Reviewed By: @ptarjan

Differential Revision: D1099088
fastcgi headers need unmangling

Headers as mangled in the FastCGI protocol, but the `getHeader()` method on `Transport` assumed the header was the original value.

Closes #1359

Reviewed By: @alexmalyshev

Differential Revision: D1098954
Fixes to parse_ini_(file|string)

Do a bunch of fixes in the parse_ini lexer and parser

Closes: #1320
Closes: #825
Closes: #838

Reviewed By: @ptarjan

Differential Revision: D970513
Disable flaky zend test

Reviewed By: @bertmaher

Differential Revision: D1100222
Fix RegSet::size() so it counts all the bits

builtin_popcount() silently truncated to int.

Reviewed By: @oyamauchi

Differential Revision: D1100154
make valgrind work

It doesn't always give us 4 bit aligned memory

Reviewed By: @edwinsmith

Differential Revision: D1060964
Add INI generator and generated files

Differential Revision: D1100639