[PATCH] D99901: [Driver][test] Test intended target only

2021-09-10 Thread Nico Weber via Phabricator via cfe-commits
thakis added inline comments. Comment at: clang/test/Driver/nostdincxx.cpp:4 // RUN: not %clangxx -nostdlibinc %s 2>&1 | FileCheck %s -// RUN: not %clangxx -fsyntax-only -nostdinc -nostdinc++ %s 2>&1 | FileCheck /dev/null --implicit-check-not=-Wunused-command-line-argument +//

[PATCH] D99901: [Driver][test] Test intended target only

2021-05-28 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added inline comments. Comment at: clang/test/Driver/nostdincxx.cpp:4 // RUN: not %clangxx -nostdlibinc %s 2>&1 | FileCheck %s -// RUN: not %clangxx -fsyntax-only -nostdinc -nostdinc++ %s 2>&1 | FileCheck /dev/null --implicit-check-not=-Wunused-command-li

[PATCH] D99901: [Driver][test] Test intended target only

2021-05-28 Thread Jinsong Ji via Phabricator via cfe-commits
jsji added a comment. In D99901#2787034 , @ted wrote: > Why would we not want to support this on other targets, like Arm? Fixes for > AIX shouldn't impact other things. Feel free to update the triple to include arm , as long as the tests focus on gnu to

[PATCH] D99901: [Driver][test] Test intended target only

2021-05-28 Thread Ted Woodward via Phabricator via cfe-commits
ted added a comment. Why would we not want to support this on other targets, like Arm? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99901/new/ https://reviews.llvm.org/D99901 ___ cfe-commits mailing lis

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-07 Thread Jinsong Ji via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa723310b419b: [Driver][test] Test intended target only (authored by jsji). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99901/new/ https://reviews.llvm.org

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-06 Thread Jinsong Ji via Phabricator via cfe-commits
jsji updated this revision to Diff 335500. jsji added a comment. Remove typo. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99901/new/ https://reviews.llvm.org/D99901 Files: clang/test/Driver/nostdincxx.cpp Index: clang/test/Driver/nostdincxx.

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-06 Thread Jinsong Ji via Phabricator via cfe-commits
jsji added inline comments. Comment at: clang/test/Driver/nostdincxx.cpp:4 // RUN: not %clangxx -nostdlibinc %s 2>&1 | FileCheck %s -// RUN: not %clangxx -fsyntax-only -nostdinc -nostdinc++ %s 2>&1 | FileCheck /dev/null --implicit-check-not=-Wunused-command-line-argument +// RU

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-06 Thread Jinsong Ji via Phabricator via cfe-commits
jsji updated this revision to Diff 335499. jsji added a comment. Use -arch instead Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99901/new/ https://reviews.llvm.org/D99901 Files: clang/test/Driver/nostdincxx.cpp Index: clang/test/Driver/nostdi

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-05 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/test/Driver/nostdincxx.cpp:4 // RUN: not %clangxx -nostdlibinc %s 2>&1 | FileCheck %s -// RUN: not %clangxx -fsyntax-only -nostdinc -nostdinc++ %s 2>&1 | FileCheck /dev/null --implicit-check-not=-Wunused-command-line-argument +//

[PATCH] D99901: [Driver][test] Test intended target only

2021-04-05 Thread Jinsong Ji via Phabricator via cfe-commits
jsji created this revision. jsji added a reviewer: MaskRay. jsji requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. 6fe7de90b9e4e466a5c2baadafd5f72d3203651d changed GNU to