[PATCH] D90063: [AIX] Also error on -G for link-only step

2020-10-26 Thread Xiangling Liao via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3d4aebbb9d99: [AIX] Also error on -G for link-only step (authored by Xiangling_L). Changed prior to commit: https://reviews.llvm.org/D90063?vs=300391&id=300782#toc Repository: rG LLVM Github Monorepo

[PATCH] D90063: [AIX] Also error on -G for link-only step

2020-10-26 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast accepted this revision. hubert.reinterpretcast added a comment. This revision is now accepted and ready to land. LGTM; thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90063/new/ https://reviews.llvm.org/D90063 ___

[PATCH] D90063: [AIX] Also error on -G for link-only step

2020-10-23 Thread Xiangling Liao via Phabricator via cfe-commits
Xiangling_L updated this revision to Diff 300391. Xiangling_L added a comment. Fixed the duplication in the testcase; CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90063/new/ https://reviews.llvm.org/D90063 Files: clang/lib/Driver/Driver.cpp clang/lib/Driver/ToolChains/Clang.cpp

[PATCH] D90063: [AIX] Also error on -G for link-only step

2020-10-23 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added inline comments. Comment at: clang/test/Driver/aix-err-options.c:14-15 + // RUN: %clang -target powerpc64-ibm-aix-xcoff -### -S -emit-llvm -G 0 2>&1 %s | \ // RUN: FileCheck --check-prefix=CHECK64 %s +// RUN: %clang -target powerpc64-ibm-aix-xcof

[PATCH] D90063: [AIX] Also error on -G for link-only step

2020-10-23 Thread Xiangling Liao via Phabricator via cfe-commits
Xiangling_L created this revision. Xiangling_L added reviewers: hubert.reinterpretcast, daltenty, jasonliu. Herald added subscribers: cfe-commits, kbarton, nemanjai. Herald added a project: clang. Xiangling_L requested review of this revision. The change in [[ https://reviews.llvm.org/D89897 | [AI