Don't force inlining for msvc targets.
authorRalph Giles <giles@mozilla.com>
Thu, 21 Dec 2017 21:48:55 +0000 (13:48 -0800)
committerRalph Giles <giles@mozilla.com>
Thu, 21 Dec 2017 21:55:18 +0000 (13:55 -0800)
commit117893a7177b4f7a63ad3bdf367980656f4ba556
tree000fa2530ef0afe4548aba9e33e5d48a7bcd525e
parent7a245adb18b8237da8bdb9ae5b7217ec047fb34a
Don't force inlining for msvc targets.

INLINE is defined as __forceinline for vs* configs, but is the
normal, compiler-discretion inline for gcc/clang configs. This
makes many functions very large when building for windows targets,
much larger than they are elsewhere.

Use '__inline' as a consistent definition to get consistent function
sizes. Although Visual Studio documentation says that 'inline' is
only available in C+ code. This is probably incorrect, since Visual
Studio 2017 accepts C99 'inline' even when passed /TC. Nevertheless,
this commit uses the recommended '__inline' for consistency.

Thanks to David Major for the diagnosis.

Change-Id: Ib0b31a3afcea77822c84fe3c6cd452add66d825a
configure