|
mark.benvenuto I was misled by the comment in SConstruct not matching the code:
# Currently we only use tcmalloc on windows and linux x86_64. Other
|
# linux targets (power, s390x, arm) do not currently support tcmalloc.
|
#
|
# Normalize the allocator option and store it in the Environment. It
|
# would be nicer to use SetOption here, but you can't reset user
|
# options for some strange reason in SCons. Instead, we store this
|
# option as a new variable in the environment.
|
if get_option('allocator') == "auto":
|
if env.TargetOSIs('windows') or \
|
env.TargetOSIs('linux'):
|
env['MONGO_ALLOCATOR'] = "tcmalloc"
|
else:
|
env['MONGO_ALLOCATOR'] = "system"
|
Based on the code, we're making sure tcmalloc is ported to all Linux targets, so the comment should be updated. Given that, I agree we don't need to update the RPM so this ticket can be closed.
|