Skip to content

Commit

Permalink
[clang][bpf] Fix invalid RUN lines in stack protector warning test (#…
Browse files Browse the repository at this point in the history
…65251)

2 of them were missing the ":" on the end.

Adding them broke the test so I had to add a new prefix just for the
warning runs only.

I manually checked the first RUNs and there is no warning emitted, as
expected.
  • Loading branch information
DavidSpickett committed Sep 14, 2023
1 parent 4f1f171 commit 7e6a5fe
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions clang/test/CodeGen/bpf-stack-protector.c
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
// REQUIRES: bpf-registered-target

// RUN %clang -target bpf -S -emit-llvm -o - %s -fno-stack-protector 2>&1 \
// RUN | FileCheck -check-prefix=OFF -check-prefix=COMMON %s
// RUN: %clang -target bpf -S -emit-llvm -o - %s -fno-stack-protector 2>&1 \
// RUN: | FileCheck -check-prefix=OFF -check-prefix=COMMON %s

// RUN: %clang -target bpf -S -emit-llvm -o - %s -fstack-protector 2>&1 \
// RUN: | FileCheck -check-prefix=ON -check-prefix=COMMON %s
// RUN: | FileCheck -check-prefix=ON -check-prefix=COMMON -check-prefix=WARNING %s

// RUN: %clang -target bpf -S -emit-llvm -o - %s -fstack-protector-all 2>&1 \
// RUN: | FileCheck -check-prefix=ALL -check-prefix=COMMON %s
// RUN: | FileCheck -check-prefix=ALL -check-prefix=COMMON -check-prefix=WARNING %s

// RUN: %clang -target bpf -S -emit-llvm -o - %s -fstack-protector-strong 2>&1 \
// RUN: | FileCheck -check-prefix=STRONG -check-prefix=COMMON %s
// RUN: | FileCheck -check-prefix=STRONG -check-prefix=COMMON -check-prefix=WARNING %s

typedef __SIZE_TYPE__ size_t;

Expand All @@ -22,7 +22,7 @@ char *strcpy(char *s1, const char *s2);
// ON: warning: ignoring '-fstack-protector'
// ALL: warning: ignoring '-fstack-protector-all'
// STRONG: warning: ignoring '-fstack-protector-strong'
// COMMON-SAME: option as it is not currently supported for target 'bpf'
// WARNING-SAME: option as it is not currently supported for target 'bpf'

// COMMON: define {{.*}}void @test1(ptr noundef %{{[0-9a-z]+}}) #[[A:.*]] {
void test1(const char *msg) {
Expand Down

0 comments on commit 7e6a5fe

Please sign in to comment.