mirror of
https://github.com/capstone-engine/llvm-capstone.git
synced 2025-01-03 06:10:23 +00:00
327894859c
Jenkins sometimes starts a new working directory by appending @2 (or
incrementing the number if the @n suffix is already there). This causes
several clang tests to fail as:
s@INPUT_DIR@%/S/Inputs@g
gets expanded to the invalid:
s@INPUT_DIR@/path/to/workdir@2/Inputs@g
~~~~~~~~~~
where the part marked with ~'s is interpreted as the flags. These are
invalid and the test fails.
Previous fixes simply exchanged the @ character for another like | but
that's just moving the problem. Address it by adding an expansion that
escapes the @ character we're using as a delimiter as well as other magic
characters in the replacement of sed's s@@@.
There's still room for expansions to cause trouble though. One I ran into
while testing this was that having a directory called foo@bar causes lots
of `CHECK-NOT: foo` directives to match. There's also things like
directories containing `\1`
41 lines
2.0 KiB
Objective-C
41 lines
2.0 KiB
Objective-C
// RUN: rm -rf %t
|
|
// RUN: mkdir %t
|
|
|
|
// RUN: %hmaptool write %S/Inputs/double-quotes/a.hmap.json %t/a.hmap
|
|
// RUN: %hmaptool write %S/Inputs/double-quotes/x.hmap.json %t/x.hmap
|
|
|
|
// RUN: sed -e "s@TEST_DIR@%{/S:regex_replacement}/Inputs/double-quotes@g" \
|
|
// RUN: %S/Inputs/double-quotes/z.yaml > %t/z.yaml
|
|
|
|
// The output with and without modules should be the same
|
|
|
|
// RUN: %clang_cc1 \
|
|
// RUN: -I %t/x.hmap -iquote %t/a.hmap -ivfsoverlay %t/z.yaml \
|
|
// RUN: -F%S/Inputs/double-quotes -I%S/Inputs/double-quotes \
|
|
// RUN: -Wquoted-include-in-framework-header -fsyntax-only %s -verify
|
|
|
|
// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache \
|
|
// RUN: -I %t/x.hmap -iquote %t/a.hmap -ivfsoverlay %t/z.yaml \
|
|
// RUN: -F%S/Inputs/double-quotes -I%S/Inputs/double-quotes \
|
|
// RUN: -Wquoted-include-in-framework-header -fsyntax-only %s \
|
|
// RUN: 2>%t/stderr
|
|
|
|
// The same warnings show up when modules is on but -verify doesn't get it
|
|
// because they only show up under the module A building context.
|
|
// RUN: FileCheck --input-file=%t/stderr %s
|
|
// CHECK: double-quoted include "A0.h" in framework header, expected angle-bracketed instead
|
|
// CHECK: double-quoted include "B.h" in framework header, expected angle-bracketed instead
|
|
// CHECK: double-quoted include "B.h" in framework header, expected angle-bracketed instead
|
|
|
|
#import "A.h"
|
|
#import <Z/Z.h>
|
|
|
|
// Make sure we correctly handle paths that resemble frameworks, but aren't.
|
|
#import "NotAFramework/Headers/Headers/Thing1.h"
|
|
|
|
int bar() { return foo(); }
|
|
|
|
// expected-warning@Inputs/double-quotes/A.framework/Headers/A.h:1{{double-quoted include "A0.h" in framework header, expected angle-bracketed instead}}
|
|
// expected-warning@Inputs/double-quotes/A.framework/Headers/A.h:2{{double-quoted include "B.h" in framework header, expected angle-bracketed instead}}
|
|
// expected-warning@Inputs/double-quotes/flat-header-path/Z.h:1{{double-quoted include "B.h" in framework header, expected angle-bracketed instead}}
|