Summary: I want to use TestLogHandler::getMessageValues with a const TestLogHandler reference. Mark getMessageValues as const so it can be called on a const TestLogHandler. getMessageValues doesn't mutate class state, so this should be safe.

Reviewed By: chadaustin

Differential Revision: D9815552

fbshipit-source-id: 275625d1a001fe536586229228eb2c9eee7e6330

Summary: The pthread library we were using is getting to be annoying so just implement things ourself.

Reviewed By: yfeldblum

Differential Revision: D9777661

fbshipit-source-id: fb53d0d369ed27a869e9f6285a548778c3492f2e

Summary:
It first looks for the name, then looks for a section with that
offset. But strings can be overlaid; eg .rela.debug_info and .debug_info
can be overlapped, so we won't find .debug_info at all.

Instead, just iterate the sections checking to see if the name matches
the one we're looking for.

Reviewed By: ricklavoie, palmtenor

Differential Revision: D9809817

fbshipit-source-id: b165719c6f9004ecc964096677114bce357ece30

Summary:
Pull Request resolved: https://github.com/facebook/folly/pull/932

As title.

Reviewed By: yfeldblum

Differential Revision: D9817328

fbshipit-source-id: 261c68f21316e91f3ed752c4aeab9e9c95dfe16d

Summary:
toAppend support DoubleToStringConverter::SHORTEST_SINGLE for float type.
Pull Request resolved: https://github.com/facebook/folly/pull/928

Reviewed By: yfeldblum

Differential Revision: D9790564

Pulled By: Orvid

fbshipit-source-id: b38d6542518aa0ed7f2b0e222e9870bb5f3a51e5

Summary: As title.

Reviewed By: JohnRambo

Differential Revision: D9820130

fbshipit-source-id: 58128c561825681d463759da33be94b861057f2e

Summary:
The `__cpuid()` and `__cpuidex()` instructions are only available on
x86 and x64 https://fburl.com/ejw47bcl.

Reported in zstd [Issue#1313](https://github.com/facebook/zstd/issues/1313).

Reviewed By: Orvid

Differential Revision: D9800189

fbshipit-source-id: 8e846d2220e544b16547db16d21c7c7b0e21cefb

Summary: MSVC wants template arguments to have exact same names in forward declarations.

Reviewed By: yfeldblum

Differential Revision: D9780751

fbshipit-source-id: 18cc529603c128e4c5d5a20bc61e0e44c0d0240c

Summary:
- GLock<SpinLock> -> GoogleSpinLockAdapter
- Duplicated BENCH_BASE -> BENCH_REL

Reviewed By: yfeldblum

Differential Revision: D9803057

fbshipit-source-id: a69961f6d429eadca6feaacd0d5d999b55d13300

Summary:
`align()` was private, so the converting construct didn't have access
to `other.align()`.

Reviewed By: yfeldblum

Differential Revision: D9785137

fbshipit-source-id: e64a66eccd8a8b28db27873c0aaa7305b642c926

Summary: [Folly] Fix weird formatting of `break` after switch-case-close-brace.

Reviewed By: Orvid

Differential Revision: D9796573

fbshipit-source-id: 5f392e9fcf33be24acb9fc7a79bdf9ca846fd33c

Summary:
[Folly] Find `libiberty` headers in subdir also.

Fixes #792.

Reviewed By: simpkins

Differential Revision: D9330225

fbshipit-source-id: 6a9c3aaf4e76e3b5c8947cb15d0b43ed6cbcc778

Summary:
Warning: https://msdn.microsoft.com/en-us/library/cxdxz3x6.aspx

Provide matching placement delete operator to avoid the false-positive MSVC warning about memory leak when exception is thrown inside the constructor.

Reviewed By: yfeldblum

Differential Revision: D9773561

fbshipit-source-id: e6c4ab1d230e567491271c4cc2c23cd98469e672

Summary: No semantic changes

Reviewed By: yfeldblum

Differential Revision: D9788374

fbshipit-source-id: e619bc19d6064899ce61acc6b67b8b8a8f92101d

Summary: [Folly] Revise the `kSelectInByte` comments - hopefully be clearer and include an example.

Reviewed By: luciang

Differential Revision: D9780161

fbshipit-source-id: 6e9b5798afed9fe161b527ab597852232c278564

Summary: [Folly] Compute `kSelectInByte` table at compile time rather than hardcoding the resulting precomputed table.

Reviewed By: luciang

Differential Revision: D9627807

fbshipit-source-id: 01fbb07dd14aac4b84ad72bac10a12d3657be589

Summary:
Currently malloc(goodMallocSize(x)) will succeed when x is larger
than the maximum possible allocation supported by jemalloc.  This is
not desirable.  Currently goodMallocSize(x) returns 0 if x is too large,
but 0 is a valid argument for malloc and causes a 1 byte allocation.
This diff causes goodMallocSize(x) to return x in that case, which will
cause the subsequent malloc to fail.  The caller may catch that failure
or crash immediately, but those are both preferrable to returning a
pointer to a valid 1-byte allocation.

Reviewed By: al13n321

Differential Revision: D9778137

fbshipit-source-id: 3eb8da72d6240b28da85483f0d91653f5e04b333

Summary:
[Folly] Refactor `StaticSingletonManager`.

* Move implementation to `.cpp`.
* Remove all `#include`s possible from `.h`.
* Control inlining.

Differential Revision: D9734721

fbshipit-source-id: 82a1cda187674a6bb62e14a80431aeb07c65156c

Summary:
[Folly] A unit-test for the `kSelectInByte` table.

The table essentially contains an encoding of every byte value. So we add a unit-test which decodes from the table, confirming that each encoded byte decodes back to the original.

Also add helpers to the test file to ease viewing disassembly of select64 variants.

Reviewed By: ot

Differential Revision: D9628409

fbshipit-source-id: 61cebaff4e9aff223a06194f7440eb3580a9273d

Summary: [Folly] Test cases for `StaticSingletonManager`.

Differential Revision: D9735443

fbshipit-source-id: e922d7742c12e6ca7725dc7782cc683cf71fb6a7

Summary: Add SpinLock from google/base/spinlock.h to the small lock benchmark

Reviewed By: yfeldblum

Differential Revision: D9771633

fbshipit-source-id: 3cf01866ec6cd4a4a5d8c2fb4274ca20a6f545dd

Summary: getApplicationProtocol should probably be const.

Reviewed By: knekritz, mingtaoy

Differential Revision: D9772305

fbshipit-source-id: c648f9307214258639616e2f4ff3a90dfbf9484b

Summary: This export the 16 times stack size for ASAN TSAN into a config as @[655434592:Andrii Grynenko] suggested

Reviewed By: yfeldblum

Differential Revision: D9685364

fbshipit-source-id: 4c76e99ba10d0fad65dd0dc30eeb003b25514c43

Summary: Use checkedMalloc everywhere in IOBuf code.

Reviewed By: yfeldblum

Differential Revision: D9763962

fbshipit-source-id: d08046b26bf0dd3be5067ad9e0706647fdcf1e83

Summary:
IOBuf::createCombined was derefencing a nullptr in case of allocation failure.
This patches fixes this issue by throwing std::bad_alloc if malloc() returns a
nullptr.

Reviewed By: yfeldblum

Differential Revision: D9763331

fbshipit-source-id: 7d9b9265079e74f5c63c8c8809099de97342f54d

Summary: Typo

Reviewed By: yfeldblum

Differential Revision: D9755749

fbshipit-source-id: b0295c12353a256e4462dfacf1bafce09c09c598

Summary:
Provide an option to force digest updates in BufferedStat. This
is useful in tests when one can't easily mock the clock.

Reviewed By: yfeldblum

Differential Revision: D9732899

fbshipit-source-id: e6a9c01d97a156bdded58fc01d8389bbf9edbbf7

Summary:
4d234b99 introduces `folly/detail/Futex-inl.h`. This patch adds it to `Makefile.am` so it gets installed when building using autotools.
Pull Request resolved: https://github.com/facebook/folly/pull/926

Reviewed By: yfeldblum

Differential Revision: D9757810

Pulled By: Orvid

fbshipit-source-id: 42bdaf292d9ca5e47c01d39381c5ebd4941dcb2a

Summary: Have AsyncSSLSocket use AsyncTransportCertificate interfaces properly

Reviewed By: mingtaoy

Differential Revision: D9752031

fbshipit-source-id: c65c0b808d82843bf1111bb650fe140ac98723b8

Summary:
F14MapTest already had properly relaxed the post-condition for
fallback maps after a move-clear and reserve; this diff applies the same
relaxation to F14SetTest.

Reviewed By: shixiao

Differential Revision: D9756311

fbshipit-source-id: 8285bb05023d923c4a7c4e014c3668bdaed591c4

Summary: Separates variants of Future::then that take differently parameterised lambdas. This will allow separately deprecating and then deleting them without having to solve all at once.

Reviewed By: yfeldblum

Differential Revision: D9512097

fbshipit-source-id: 1b6c0946f0fa38cf8427c66fe1bef47315875e58

Summary:
[Folly] Smaller implementation of `to_ordering`.

Shorter source code, shorter bytecode, no branches/conditionals.

Reviewed By: Orvid

Differential Revision: D9733375

fbshipit-source-id: a2b2a8263b5de2264d3068a424419d63e5b27228

Summary:
[Folly] Add `FOLLY_HAS_EXCEPTIONS` preprocessor symbol.

Add checks for:
* `__cpp_exceptions` and `__has_feature(cxx_exceptions)`.
* MSVC platform.

But avoiding a single unwieldy expression.

And use it in `throw_exception`.

Reviewed By: mnovakovic

Differential Revision: D9732924

fbshipit-source-id: 4ac00e2d84007dfe8307f0e861ab2edfbde80fff

Summary:
[Folly] Assorted partial applications of `clang-format`.

Some parts of the code are explicitly opted out where `clang-format` produces poor results, and for other reasons.

Future passes may remove current opt-outs.

Reviewed By: Orvid, igorsugak

Differential Revision: D9727723

fbshipit-source-id: d191ae51f535335402d5e083c3368d83779b7f6e

Summary: Disable recording highWaterMark for asan builds. I discovered this while using getHighWatermark method on test runs (build with ASAN).  It gave bogus value (>90MB), when stack size was only 24k.

Reviewed By: andriigrynenko

Differential Revision: D9687204

fbshipit-source-id: 7ca311acb6f310694850f065febc3998a11d036a

Summary: [Folly] Split the `FBString.testFixedBugs` test case into a test case per bug.

Reviewed By: Orvid

Differential Revision: D9727569

fbshipit-source-id: b319286031ded00451a1e960c0d450e7cfc4c362

Summary: Fix `throw_exception` with ` -fno-exceptions -Werror=unused-parameter`.

Reviewed By: yfeldblum

Differential Revision: D9693907

fbshipit-source-id: d308147b7d69cf091a9f839fda8be3464aa94b9a

Fix: folly/experimental/coro/test/CoroBenchmarkAllocator.cpp:363:20: error: 'Recursion' has not been declared

Summary: Fix: folly/experimental/coro/test/CoroBenchmarkAllocator.cpp:363:20: error: 'Recursion' has not been declared

Reviewed By: yfeldblum

Differential Revision: D9727088

fbshipit-source-id: 91447187bc6d34b965d5f8c2cedd66950b1fe909

Summary:
This diff adds operator=(initializer_list) to the F14 fallback
implementations, relaxes the guarantees provided by visitAllocationClasses
for fallbacks (so that the current implementation can be considered
correct), enables additional tests, and avoids a bit of logic repetition.
Changes should only be made to the fallback mode and tests, and this diff
should not change the set of platforms for which fallback mode is enabled.

Reviewed By: shixiao

Differential Revision: D9664700

fbshipit-source-id: 1f3fc6f0c48009bb3e5fd249506b42f54d93df43

Summary:
Remove bogus `configure` argument from cmake commands which is likely an
artefact from automake migration. The correct invocation of CMake is

```
cmake [<options>] {<path-to-source> | <path-to-existing-build>}
```

Reviewed By: Orvid

Differential Revision: D9721030

fbshipit-source-id: 95da844d3292cbf2acbec85753499f3603567e08