[PATCH] D95660: [NFC] Disallow unused prefixes under clang/test/Driver

Mircea Trofin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 29 17:20:45 PST 2021


mtrofin added inline comments.


================
Comment at: clang/test/Driver/rocm-device-libs.cl:82
 // RUN:   %s \
-// RUN: 2>&1 | FileCheck  --check-prefixes=COMMON,COMMON-UNSAFE,GFX803,WAVE64 %s
+// RUN: 2>&1 | FileCheck  --check-prefixes=COMMON,GFX803,WAVE64 %s
 
----------------
MaskRay wrote:
> Since you changing the lines, consider indenting `2>&1`
> 
> The most common style is to place `2>&1 | \` on the previous line.
done


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95660



More information about the cfe-commits mailing list