This is an archive of the discontinued LLVM Phabricator instance.

Fix option desc in FunctionAttrs; NFC
ClosedPublic

Authored by weimingz on Dec 28 2015, 5:10 PM.

Reviewers

jmolloy

Commits

Summary

The example in desc should match with actual option name

weimingz retitled this revision from to Fix option desc in FunctionAttrs; NFC.

weimingz updated this object.

weimingz set the repository for this revision to rL LLVM.

Thanks Weiming - it looks like the comment did not survive a variable rename.

LGTM.

This revision is now accepted and ready to land.Jan 6 2016, 6:08 AM

rebased

Path

Size

lib/

Transforms/

IPO/

2 lines

	Show All 16 Lines
	using namespace llvm;			using namespace llvm;

	#define DEBUG_TYPE "forceattrs"			#define DEBUG_TYPE "forceattrs"

	static cl::list<std::string>			static cl::list<std::string>
	ForceAttributes("force-attribute", cl::Hidden,			ForceAttributes("force-attribute", cl::Hidden,
	cl::desc("Add an attribute to a function. This should be a "			cl::desc("Add an attribute to a function. This should be a "
	"pair of 'function-name:attribute-name', for "			"pair of 'function-name:attribute-name', for "
	"example -force-add-attribute=foo:noinline. This "			"example -force-attribute=foo:noinline. This "
	"option can be specified multiple times."));			"option can be specified multiple times."));

	static Attribute::AttrKind parseAttrKind(StringRef Kind) {			static Attribute::AttrKind parseAttrKind(StringRef Kind) {
	return StringSwitch<Attribute::AttrKind>(Kind)			return StringSwitch<Attribute::AttrKind>(Kind)
	.Case("alwaysinline", Attribute::AlwaysInline)			.Case("alwaysinline", Attribute::AlwaysInline)
	.Case("builtin", Attribute::Builtin)			.Case("builtin", Attribute::Builtin)
	.Case("cold", Attribute::Cold)			.Case("cold", Attribute::Cold)
	.Case("convergent", Attribute::Convergent)			.Case("convergent", Attribute::Convergent)
	▲ Show 20 Lines • Show All 88 Lines • Show Last 20 Lines