[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Steven Wan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 27 11:49:38 PST 2022


stevewan added inline comments.


================
Comment at: clang/test/ClangScanDeps/headerwithdirname.cpp:16
 
-// CHECK: headerwithdirname_input.o
+// CHECK: headerwithdirname_input{{.*}}{{.o|.s}}
 // CHECK-NEXT: headerwithdirname_input.cpp
----------------
Jake-Egan wrote:
> jansvoboda11 wrote:
> > Why is `{{.*}}` necessary?
> It's to match the random characters between "input" and the file extension. The output on AIX looks like this:
> `/tmp/headerwithdirname_input-ffd608.s: \`
To be clear, this isn't AIX specific behaviour. We get the temp assembly file when `-fno-integrated-as` is on. 

Maybe slightly more accurate is,
```
// CHECK: headerwithdirname_input{{\.o|.*\.s}}
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118152/new/

https://reviews.llvm.org/D118152



More information about the cfe-commits mailing list