This is an archive of the discontinued LLVM Phabricator instance.

[zorg] Pass "-DLLVM_POLLY_LINK_INTO_TOOLS=ON" in getPollyBuildFactory
ClosedPublic

Authored by efriedma on Jan 13 2020, 1:58 PM.

Reviewers

gkistanova
bollu

Commits

Summary

The default here was changed recently, and the aosp-O3-polly-before-vectorizer-unprofitable builder assumes polly is linked into clang.

LGTM

This revision is now accepted and ready to land.Jan 13 2020, 7:28 PM

This revision was automatically updated to reflect the committed changes.

It looks like the build isn't passing the new flag to cmake. Does the buildbot need to be restarted?

Yes, restart is required. It's planned tonight.

Path

Size

zorg/

buildbot/

builders/

1 line

Show First 20 Lines • Show All 70 Lines • ▼ Show 20 Lines	f.addStep(ShellCommand(name='clean-build-dir',
doStepIf=cleanBuildRequested))		doStepIf=cleanBuildRequested))

# Create configuration files with cmake		# Create configuration files with cmake
cmakeCommand = ["cmake", "../%s/llvm" % llvm_srcdir,		cmakeCommand = ["cmake", "../%s/llvm" % llvm_srcdir,
"-DCMAKE_COLOR_MAKEFILE=OFF",		"-DCMAKE_COLOR_MAKEFILE=OFF",
"-DPOLLY_TEST_DISABLE_BAR=ON",		"-DPOLLY_TEST_DISABLE_BAR=ON",
"-DPOLLY_ENABLE_GPGPU_CODEGEN=ON",		"-DPOLLY_ENABLE_GPGPU_CODEGEN=ON",
"-DCMAKE_BUILD_TYPE=Release",		"-DCMAKE_BUILD_TYPE=Release",
		"-DLLVM_POLLY_LINK_INTO_TOOLS=ON",
"-DLLVM_ENABLE_PROJECTS=%s" % ";".join(f.depends_on_projects),		"-DLLVM_ENABLE_PROJECTS=%s" % ";".join(f.depends_on_projects),
] + cmake_install + extraCmakeArgs		] + cmake_install + extraCmakeArgs
f.addStep(ShellCommand(name="cmake-configure",		f.addStep(ShellCommand(name="cmake-configure",
command=cmakeCommand,		command=cmakeCommand,
haltOnFailure=False,		haltOnFailure=False,
description=["cmake configure"],		description=["cmake configure"],
workdir=llvm_objdir,		workdir=llvm_objdir,
env=merged_env))		env=merged_env))
▲ Show 20 Lines • Show All 43 Lines • Show Last 20 Lines