test/Driver/malign-double.c - Issue 1323073010: Use LLVM args to handle -malign-double option

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: test/Driver/malign-double.c

Issue 1323073010: Use LLVM args to handle -malign-double option (Closed) Base URL: https://chromium.googlesource.com/a/native_client/pnacl-clang.git@master

Patch Set: Created 5 years, 3 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: test/Driver/malign-double.c

diff --git a/test/Driver/malign-double.c b/test/Driver/malign-double.c

index 81c087909223042c62a80c8793730ebb56791243..73c52fe9912927082c52af7c8daee15e8f7aecdc 100644

--- a/test/Driver/malign-double.c

+++ b/test/Driver/malign-double.c

@@ -1,13 +1,12 @@

-// RUN: %clang -### -c -malign-double %s -target i686-unknown-linux 2>&1 \

+// RUN: %clang -v -c -malign-double %s -target i686-unknown-linux 2>&1 \

// RUN: | FileCheck --check-prefix=CHECK-I686 %s

// CHECK-I686: -cc1

-// CHECK-I686: +align-double

-// CHECK-I686: "-mllvm" "-malign-double"

+// CHECK-I686: -mllvm -malign-double

+// CHECK-I686-NOT: backend data layout 'e-m:e-p:32:32-i64:64-f80:32-n8:16:32-S128' does not match expected target description 'e-m:e-p:32:32-f64:32:64-f80:32-n8:16:32-S128'

Derek Schuff 2015/09/09 22:45:56 could we maybe reduce this so it still fires even

Petr Hosek 2015/09/09 23:00:23 Regex to the rescue!

-// RUN: %clang -### -c -malign-double %s -target x86-64-unknown-linux 2>&1 \

-// RUN: | FileCheck -check-prefix=CHECK-X86_64 %s

+// RUN: %clang -v -c -malign-double %s -target x86_64-unknown-linux 2>&1 \

+// RUN: | FileCheck --check-prefix=CHECK-X86_64 %s

// CHECK-X86_64: -cc1

-// CHECK-X86_64-NOT: +align-double

-// CHECK-X86_64-NOT: "-mllvm" "-malign-double"

+// CHECK-X86_64: -mllvm -malign-double

« lib/Basic/Targets.cpp ('K') | « lib/Frontend/CompilerInvocation.cpp ('k') | no next file » | no next file with comments »