This is an archive of the discontinued LLVM Phabricator instance.

Differential D56634

[MSP430] Recoginze '{' as a line separator
ClosedPublic

Authored by krisb on Jan 12 2019, 3:00 AM.

Download Raw Diff

Details

Reviewers

asl

Diff Detail

Repository

rL LLVM

Build Status

Buildable 26747
Build 26746: arc lint + arc unit

Event Timeline

krisb created this revision.Jan 12 2019, 3:00 AM

Herald added a subscriber: llvm-commits. · View Herald TranscriptJan 12 2019, 3:00 AM

Harbormaster completed remote builds in B26747: Diff 181431.Jan 12 2019, 3:00 AM

asl accepted this revision.Jan 15 2019, 11:52 AM

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

Committed as r351233, thanks!

Revision Contents

Path

Size

lib/

Target/

MSP430/

MCTargetDesc/

MSP430MCAsmInfo.cpp

1 line

test/

MC/

MSP430/

msp430-separator.s

15 lines

Diff 181431

lib/Target/MSP430/MCTargetDesc/MSP430MCAsmInfo.cpp

	Show All 14 Lines
	using namespace llvm;			using namespace llvm;

	void MSP430MCAsmInfo::anchor() { }			void MSP430MCAsmInfo::anchor() { }

	MSP430MCAsmInfo::MSP430MCAsmInfo(const Triple &TT) {			MSP430MCAsmInfo::MSP430MCAsmInfo(const Triple &TT) {
	CodePointerSize = CalleeSaveStackSlotSize = 2;			CodePointerSize = CalleeSaveStackSlotSize = 2;

	CommentString = ";";			CommentString = ";";
				SeparatorString = "{";

	AlignmentIsInBytes = false;			AlignmentIsInBytes = false;
	UsesELFSectionDirectiveForBSS = true;			UsesELFSectionDirectiveForBSS = true;
	}			}

test/MC/MSP430/msp430-separator.s

This file was added.

				; RUN: llvm-mc -triple msp430 < %s \| FileCheck %s

				; MSP430 supports multiple assembly statements on the same line
				; separated by a '{' character.

				; Check that the '{' is recognized as a line separator and
				; multiple statements correctly parsed.

				_foo:
				; CHECK: foo
				; CHECK: add r10, r11
				; CHECK-NEXT: call r11
				; CHECK-NEXT: mov r11, 2(r1)
				add r10, r11 { call r11 { mov r11, 2(r1)
				ret