Skip to content

Commit

Permalink
Backed out changeset 7a3882f27aa3 (bug 1341525) for failing win Sa bu…
Browse files Browse the repository at this point in the history
…ilds on a CLOSED TREE

--HG--
extra : amend_source : dbf7c208749c67b6bb8674eb5ba55c6514f89a2d
  • Loading branch information
AndreeaPavel committed Aug 1, 2018
1 parent e02d554 commit 11f1c09
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 33 deletions.
4 changes: 0 additions & 4 deletions Makefile.in
Original file line number Diff line number Diff line change
Expand Up @@ -314,10 +314,6 @@ endif
else
ifdef CLANG_CL
maybe_clobber_profiledbuild: clean
# 32-bit PGO is currently blocked by bug 1479800
ifeq ($(CPU_ARCH),x86_64)
$(LLVM_PROFDATA) merge -o $(DEPTH)/merged.profdata $(DEPTH)/*.profraw
endif
else
maybe_clobber_profiledbuild:
$(RM) $(DIST)/bin/*.pgc
Expand Down
25 changes: 2 additions & 23 deletions build/moz.configure/toolchain.configure
Original file line number Diff line number Diff line change
Expand Up @@ -1240,14 +1240,10 @@ set_config('WRAP_SYSTEM_INCLUDES', wrap_system_includes)
set_config('VISIBILITY_FLAGS', visibility_flags)


@depends(c_compiler, check_build_environment, target)
@depends(c_compiler)
@imports('multiprocessing')
@imports(_from='__builtin__', _import='min')
def pgo_flags(compiler, build_env, target):
topobjdir = build_env.topobjdir
if topobjdir.endswith('/js/src'):
topobjdir = topobjdir[:-7]

def pgo_flags(compiler):
if compiler.type in ('gcc', 'clang'):
return namespace(
gen_cflags=['-fprofile-generate'],
Expand All @@ -1257,18 +1253,6 @@ def pgo_flags(compiler, build_env, target):
use_ldflags=['-fprofile-use'],
)

if compiler.type == 'clang-cl':
profdata = os.path.join(topobjdir, 'merged.profdata')
# 32-bit PGO is currently blocked by bug 1479800
if target.cpu == 'x86_64':
return namespace(
gen_cflags=['-fprofile-instr-generate'],
gen_ldflags=['clang_rt.profile-x86_64.lib'],
use_cflags=['-fprofile-instr-use=%s' % profdata,
'-Wno-error=profile-instr-unprofiled'],
use_ldflags=[],
)

if compiler.type == 'msvc':
num_cores = min(8, multiprocessing.cpu_count())
cgthreads = '-CGTHREADS:%s' % num_cores
Expand Down Expand Up @@ -1298,11 +1282,6 @@ set_config('PROFILE_GEN_LDFLAGS', pgo_flags.gen_ldflags)
set_config('PROFILE_USE_CFLAGS', pgo_flags.use_cflags)
set_config('PROFILE_USE_LDFLAGS', pgo_flags.use_ldflags)

llvm_profdata = check_prog('LLVM_PROFDATA', ['llvm-profdata'],
allow_missing=True)

add_old_configure_assignment('LLVM_PROFDATA', llvm_profdata)


@depends(c_compiler)
def preprocess_option(compiler):
Expand Down
7 changes: 1 addition & 6 deletions build/mozconfig.clang-cl
Original file line number Diff line number Diff line change
@@ -1,13 +1,8 @@
if test -d "$topsrcdir/clang/bin"; then
CLANG_DIR=`cd "$topsrcdir/clang/bin" ; pwd`
export PATH="${CLANG_DIR}:${PATH}"
mk_export_correct_style PATH
fi

if test -d "$topsrcdir/clang/lib/clang"; then
CLANG_LIB_DIR="$(cd $topsrcdir/clang/lib/clang/* && cd lib/windows && pwd)"
export LIB=$LIB:$CLANG_LIB_DIR
mk_export_correct_style LIB
mk_export_correct_style PATH
fi

export CC=clang-cl
Expand Down

0 comments on commit 11f1c09

Please sign in to comment.