Changed FrameEncoder::m_tld to a pointer and set it to one of Encoder's ThreadLocalData instances.

This uses less memory since m_tld isn't used in --wpp and Encoder's ThreadLocalData instances are not used in --no-wpp
Also there was a small performance increase on my system
Backed out changeset: eb011fa1d2d8
Backed out changeset: 940cec3bf0b4

This commit causes hash mismatches in vc11 x86_64 Release
mode consistently, when lft is enabled. Stack/heap corruption
likely.
encoder: rename cuCoder to analysis for better clarity

the data type of cuCoder changed from TEncCu to Analysis weeks ago
predict: inline single call of predInterBi()
predict: inline predInterUni(), getWpScaling() and simplify motionCompensation()

After this refactor, motionCompensation no longer needs the cu parameter. It
was only used to pass to another member function to gain access to cu->m_slice
which is now a member variable.

This refactor removed a number of arguments to addWeightBi and addWeightUni
which were always member variables.
predict: remove check for reallocations, comment nits

we don't do this anywhere else; there would be huge leaks if the Search object
were initialized multiple times. there's no reason to check here.
predict: combine and check allocations and return failures
predict: split weighted prediction values from WeightParam

The arguments passed to addWeightBi() and addWeightUni() are just the
"w, o, offset, shift, round" integers. They don't need the fields which were
signaled in the slice header or vice-versa.