This is an archive of the discontinued LLVM Phabricator instance.

Add type to FileSpec::PathSyntax enum.
ClosedPublic

Authored by hintonda on Nov 2 2017, 3:15 PM.

Reviewers

zturner
clayborg

Commits

Summary

Add type to FileSpec::PathSyntax enum to decrease size for
FileSpec on systems with 32 bit pointers.

Thanks to @zturner for pointing this out.

This revision is now accepted and ready to land.Nov 2 2017, 4:06 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

lldb/

trunk/

include/

lldb/

Utility/

2 lines

	Show First 20 Lines • Show All 55 Lines • ▼ Show 20 Lines
	/// to limit the scope in which to execute a command. The user rarely			/// to limit the scope in which to execute a command. The user rarely
	/// types in a full path. When the paths are already split up, it makes			/// types in a full path. When the paths are already split up, it makes
	/// it easy for us to compare only the basenames of a lot of file			/// it easy for us to compare only the basenames of a lot of file
	/// specifications without having to split up the file path each time			/// specifications without having to split up the file path each time
	/// to get to the basename.			/// to get to the basename.
	//----------------------------------------------------------------------			//----------------------------------------------------------------------
	class FileSpec {			class FileSpec {
	public:			public:
	enum PathSyntax {			enum PathSyntax : unsigned char {
	ePathSyntaxPosix,			ePathSyntaxPosix,
	ePathSyntaxWindows,			ePathSyntaxWindows,
	ePathSyntaxHostNative			ePathSyntaxHostNative
	};			};

	FileSpec();			FileSpec();

	//------------------------------------------------------------------			//------------------------------------------------------------------
	▲ Show 20 Lines • Show All 549 Lines • Show Last 20 Lines