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
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
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
__
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
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
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