Change the Profiler constructor to not necessarily force CPU affinity.

Summary: CPU affinity is needed to get monotonic real time clock ticks from rdtsc,
but that effectively disables running multiple threads in server mode.

Some profilers may not need the rdtsc data.

All existing profilers in the code base preserve their previous semantics.
Closes https://github.com/facebook/hhvm/pull/4163

Reviewed By: @JoelMarcey

Differential Revision: D1666121

Signature: t1:1666121:1415832657:60deae3e777ee3bddebd0bf97b0c4bbb3b820486
qlzcompress is not available everywhere, and there's no support for it i...

Summary: ...n OSS
Closes https://github.com/facebook/hhvm/pull/4236

Reviewed By: @JoelMarcey

Differential Revision: D1677289

Signature: t1:1677289:1415838727:13f5c39992451100d1131869f931f6c1ccb2128b
Remove magic quotes

Summary: It has beeen removed since PHP5.4
Closes https://github.com/facebook/hhvm/pull/4203

Reviewed By: @JoelMarcey

Differential Revision: D1669988

Signature: t1:1669988:1415843237:56b2bb711980d9d2e356a5e2799be290f975a155
Break regions after FCall & friends when using LLVM

Summary: When using the LLVM backend, we're going to break regions after
bytecodes that use bindcall or something similar. This is intended to be
temporary and for reasons too lengthy to go into here. I also cleaned things up
a little so we actually obey the intent of the EvalJitLLVM runtime option: 1
means Optimize translations only, while >1 means all translations.

Reviewed By: @jasone

Differential Revision: D1671586

Signature: t1:1671586:1415671170:c540b409c2d27aff6d9f00723a3d0e17fceb9e6e
Go back to individual cache props for statics

Summary: This diff is a prereq for experimenting with using individual props for instance methods when there are <N of them. It accomplishes a few things:
- Centralize the prop creation logic to make that experiment easier
- Go back to individaul props for statics. Statics almost certainly didn't contribute to the memory regression, and there tend to be only a couple per class so the extra layer was probably a negative
- Eliminates a local when using shared props in favor of emitting the int directly which should make that scenario slightly more efficient.

Reviewed By: @jano

Differential Revision: D1659983

Signature: t1:1659983:1415835257:6e1f6b6230da8b9637049b0d50087ca321e1f428
add at() to MysqlRow in hhi file

Summary: as subject

Reviewed By: @​anca

Differential Revision: D1677983

Signature: t1:1677983:1415840723:029c1dd8214a62e05edbb579c5d648255c34e5e1
unparse constants defined using define

Summary: The hack parser parses usage of "define" as a function call unless it happens to be using a String. This
unparses those correctly

Reviewed By: @jwatzman

Differential Revision: D1676521

Signature: t1:1676521:1415838137:32805e7539a9564f6e02785e7748c35e1f78fa61
merge lvalue and lvalue_foreach

Summary: A previous diff introduced lvalue_foreach inside GetLocals. This merges lvalue_foreach and lvalue
to capture lvalues. Any invalid representations that are possible should ideally be weeded out during the parsing phase.

Reviewed By: @elgenie

Differential Revision: D1678282

Signature: t1:1678282:1415846509:41c9055a003d372b0892078127896043f0870400
Pass absolute path from TranslatePath to TranslatePathRelative

Summary: When SafeFileAccess is on trying to access relative paths would always fail.
Additionally paths which failed the security check were being translated to be
the cwd, which could itself be outside the allowed directories.

Fixes #2774

Reviewed By: @JoelMarcey

Differential Revision: D1677651

Signature: t1:1677651:1415837620:50d0b15b04da5641f2f7c82a173c791e8478029f
Wait for clean shutdown on SIGHUP

Summary: HHVM is supposed to do a clean shutdown when it receives SIGHUP -
which it does when using LibEventServer - but with ProxygenServer, it spins off
an additional thread to shutdown queues, leaving the original thread to
re-raise the signal before the queues drain.  Fix this by waiting for the
servers to stop.

Reviewed By: @markw65

Differential Revision: D1675300

Signature: t1:1675300:1415808825:f77150b8cb0ea85eaf90d58095be4baeaea2d317
Fix use-after-free in the access logs

Summary: If a request put an apc string into the ServerNote
array, and then outlived all its peers, and the string was
replaced during the request, then the string could be freed
by the treadmill at the end of the request. We used to free
up the treadmill items prior to writing the access logs,
which could then fail.

Reviewed By: @ottoni

Differential Revision: D1674896
Add per kind ArrayFunc counters for ArrayTracer

Summary: Adds an array of std::atomic<uint64_t> to keep count of ArrayFunc per ArrayKind

Reviewed By: @edwinsmith

Differential Revision: D1623929
Fix duplicate unbound name errors

Summary: When naming a hint, we'd call `Env.class_name` on it twice
here, causing it to get looked up and canonicalized more than once --
and thus insert an error more than once if it was wrong, such as the
case in the github issue linked below. This is hard to test for --
`hh_server --check` will unique-ify errors, and `hh_single` will only
take the first one. Fixing the latter is beyond what I want to deal with
right now, so this manually tested fix it is.

Fixes https://github.com/facebook/hhvm/issues/3294

Reviewed By: @int3, @elgenie

Differential Revision: D1677992

Signature: t1:1677992:1415857065:675182d27592e614b5ee8864648b3367a2ef01e3
Remove main_js_comp.ml and main_hhbc_comp.ml.

Summary: Verified with@pikatchu@joelbeales and@jdelong that those files
were no longer necessary. The question arose because those files invoke
Pos.make, whose call signature I recently changed.

Reviewed By: @jdelong

Differential Revision: D1678179

Signature: t1:1678179:1415894809:ae35c4c1f6c2bc196096379e905d2faead5bb503
Require LLVM codegen when EvalJitLLVM >= 3

Summary: This will help ensure people don't accidentally regress the llvm
backend by introducing punts in code we already support.

Reviewed By: @jasone

Differential Revision: D1673489

Signature: t1:1673489:1415737138:2ea17521b6d3c917b92dd149102e5c693b7068d9
Add support for syncpoints and exceptions in the llvm backend

Summary: Both are fairly straightforward and mostly involve parsing information
out of the data sections emitted by LLVM. As far as I can tell, LLVM just emits
the exception tables into memory we give it and doesn't actually register any
of the exception information with the runtime. This means we can allocate all
data sections it asks for with malloc and free them once we extract the
information we need, rather than leaking them in globalData.

Reviewed By: @maksfb

Differential Revision: D1678102
Remove a comment that predates removing analyze()

Summary: This seems to be saying TransProfile + GuardRelaxation didn't work in
analyze().  Should we keep the EvalJitRegionSelector check or not?

Reviewed By: @swtaarrs

Differential Revision: D1651369

Signature: t1:1651369:1415219871:6c35b3497972db8290a9a1cdf98853db0ff59df4
Rename vasm's end instruction to fallthru

Summary: It conflicts with non-member use of end() on STL-style containers.

Reviewed By: @edwinsmith

Differential Revision: D1669397

Signature: t1:1669397:1415620123:5a835c9db286c4f752b75788202bc874265376dc
Simplify ArrayGet on constant arrays

Summary: I saw an IR snippet doing ArrayGets on an array we knew was static.
(Hhbbc also knew, but doesn't constant propagate anything out of minstrs, and
that'd be harder right now.)

Reviewed By: @markw65

Differential Revision: D1654328

Signature: t1:1654328:1414891382:4baf7e0dc880be238aa17b7d1eb43a5cf00f12bc
Fix some bugs in intersecting specialized types

Summary: If you intersect two arrays of different kinds you should always get
bottom, so it was being unnecessarily pessimistic.  If you intersect two
non-exact specialized classes that could be interfaces, you can't return bottom
unless you prove that there's no mutually derived classes.

Reviewed By: @swtaarrs

Differential Revision: D1669253