This is an archive of the discontinued LLVM Phabricator instance.

[ARC] Make char unsigned by default
ClosedPublic

Authored by apolyakov on Sep 3 2018, 6:41 AM.

Reviewers

tatyana-krasnukha
clayborg

Commits

Summary

This patch specifies 'char' default sign on ARC.

Good to me, if no one is confused about absent general support of ARC target. Please, add @clayborg to reviewers.

This revision is now accepted and ready to land.Sep 3 2018, 7:10 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

source/

Utility/

1 line

Show First 20 Lines • Show All 809 Lines • ▼ Show 20 Lines	bool ArchSpec::CharIsSignedByDefault() const {

case llvm::Triple::ppc:		case llvm::Triple::ppc:
case llvm::Triple::ppc64:		case llvm::Triple::ppc64:
return m_triple.isOSDarwin();		return m_triple.isOSDarwin();

case llvm::Triple::ppc64le:		case llvm::Triple::ppc64le:
case llvm::Triple::systemz:		case llvm::Triple::systemz:
case llvm::Triple::xcore:		case llvm::Triple::xcore:
		case llvm::Triple::arc:
return false;		return false;
}		}
}		}

lldb::ByteOrder ArchSpec::GetByteOrder() const {		lldb::ByteOrder ArchSpec::GetByteOrder() const {
if (m_byte_order == eByteOrderInvalid)		if (m_byte_order == eByteOrderInvalid)
return GetDefaultEndian();		return GetDefaultEndian();
return m_byte_order;		return m_byte_order;
▲ Show 20 Lines • Show All 675 Lines • Show Last 20 Lines