[PATCH] D38040: [OpenMP] Add an additional test for D34888

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 20 09:41:39 PDT 2017


tra added inline comments.


================
Comment at: test/OpenMP/target_map_codegen.cpp:4845
+///==========================================================================///
+// RUN: %clang_cc1 -DCK30 -std=c++11 -fopenmp -S -emit-llvm -fopenmp -fopenmp-targets=nvptx64-nvidia-cuda %s -o - 2>&1 | FileCheck -check-prefix=CK30 %s
+
----------------
tra wrote:
> Please break the line to make it easier to read.
Better, but the line still wraps.  I'd split it again somewhere between -emit-llvm and -fopenmp.
BTW, you have -fopenmp specified twice. Is that intentional?


Repository:
  rL LLVM

https://reviews.llvm.org/D38040





More information about the cfe-commits mailing list