This is an archive of the discontinued LLVM Phabricator instance.

Remove unused %host_cc lit pattern
ClosedPublic

Authored by thakis on Jan 2 2019, 7:36 AM.

Reviewers

Commits

Summary

It was added in r257236 but then the one use was removed in r309517. Since no test should call %host_cc, remove the pattern.

thakis edited the summary of this revision. (Show Details)

Looks like dblaikie is away for a bit. rnk, would you mind looking at this one too?

lgtm

This revision is now accepted and ready to land.Jan 3 2019, 11:21 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

test/

1 line

	Show First 20 Lines • Show All 83 Lines • ▼ Show 20 Lines


	llvm_config.use_default_substitutions()			llvm_config.use_default_substitutions()

	# Add site-specific substitutions.			# Add site-specific substitutions.
	config.substitutions.append(('%llvmshlibdir', config.llvm_shlib_dir))			config.substitutions.append(('%llvmshlibdir', config.llvm_shlib_dir))
	config.substitutions.append(('%shlibext', config.llvm_shlib_ext))			config.substitutions.append(('%shlibext', config.llvm_shlib_ext))
	config.substitutions.append(('%exeext', config.llvm_exe_ext))			config.substitutions.append(('%exeext', config.llvm_exe_ext))
	config.substitutions.append(('%host_cc', config.host_cc))


	lli_args = []			lli_args = []
	# The target triple used by default by lli is the process target triple (some			# The target triple used by default by lli is the process target triple (some
	# triple appropriate for generating code for the current process) but because			# triple appropriate for generating code for the current process) but because
	# we don't support COFF in MCJIT well enough for the tests, force ELF format on			# we don't support COFF in MCJIT well enough for the tests, force ELF format on
	# Windows. FIXME: the process target triple should be used here, but this is			# Windows. FIXME: the process target triple should be used here, but this is
	# difficult to obtain on Windows.			# difficult to obtain on Windows.
	▲ Show 20 Lines • Show All 229 Lines • Show Last 20 Lines