[PATCH] D101331: hurd: Detect libstdc++ include paths on Debian Hurd i386

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 26 16:12:43 PDT 2021


MaskRay added inline comments.


================
Comment at: clang/test/Driver/hurd.cpp:1
+// RUN: %clang -no-canonical-prefixes %s -### 2>&1 \
+// RUN:     --target=i386-pc-gnu \
----------------
I think we need -NEXT patterns to prevent unintentional new paths.

IMO `linux-cross.cpp` uses a good style and can serve as a reference.


================
Comment at: clang/test/Driver/hurd.cpp:5
+// RUN:   | FileCheck --check-prefix=CHECK %s
+// CHECK-NOT: warning:
+// CHECK: "-cc1"
----------------
The warning negative pattern doesn't really work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101331



More information about the cfe-commits mailing list