[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-10 Thread Kazushi Marukawa via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGcd5855ac3ba7: [VE] Remove -faddrsig and -fnoaddrsig tests (authored by kaz7). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION htt

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-10 Thread Kazushi Marukawa via Phabricator via cfe-commits
kaz7 added a comment. Thanks! Working on sysroot stuff now. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92996/new/ https://reviews.llvm.org/D92996 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-10 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92996/new/ https://reviews.llvm.org/D92996 __

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-10 Thread Kazushi Marukawa via Phabricator via cfe-commits
kaz7 added a comment. Thank you for additional suggestions. I'll make next patch like "optimize ve-toolchain.c and .cpp." Thank you so much! Comment at: clang/test/Driver/ve-toolchain.cpp:72 // RUN: %clangxx -### -target ve %s 2>&1 | FileCheck -check-prefix=DEFINITARRAY

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-09 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/test/Driver/ve-toolchain.cpp:72 // RUN: %clangxx -### -target ve %s 2>&1 | FileCheck -check-prefix=DEFINITARRAY %s // DEFINITARRAY: clang{{.*}} "-cc1" Most newer toolchains default to -fuse-init-array. There i

[PATCH] D92996: [VE] Remove -faddrsig and -fnoaddrsig tests

2020-12-09 Thread Kazushi Marukawa via Phabricator via cfe-commits
kaz7 created this revision. kaz7 added reviewers: MaskRay, simoll, k-ishizaka. kaz7 added projects: VE, clang. kaz7 requested review of this revision. Herald added a subscriber: cfe-commits. Remove explicitly declared -faddrsig and -fnoaddrsig option tests since those are already tested in addrsig