[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-30 Thread Nico Weber via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG61da501b6a1b: clang/test: Remove platform-linker feature (authored by thakis). Herald added a subscriber: jrtc27. Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-30 Thread Vlad Vereschaka via Phabricator via cfe-commits
vvereschaka accepted this revision. vvereschaka added a comment. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92291/new/ https://reviews.llvm.org/D92291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-30 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D92291#2423417 , @leonardchan wrote: > @phosek This shouldn't affect supporting riscv with our toolchain, right? Yes, it shouldn't have any impact. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92291/new/

[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-30 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a subscriber: phosek. leonardchan added a comment. @phosek This shouldn't affect supporting riscv with our toolchain, right? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92291/new/ https://reviews.llvm.org/D92291 ___

[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-30 Thread serge via Phabricator via cfe-commits
serge-sans-paille added a comment. If that's okay with @leonardchan in terms of RISCV validation, I'm fine with that one. Any branch removed is less path to cover! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92291/new/ https://reviews.llvm.org/D92291

[PATCH] D92291: clang/test: Remove platform-linker feature

2020-11-29 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added reviewers: serge-sans-paille, leonardchan, vvereschaka. Herald added subscribers: llvm-commits, frasercrmck, luismarques, apazos, sameer.abuasal, pzheng, s.egerton, lenary, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones,