[PATCH] D26944: Suppress -Wliblto warning when -flto is not present

2016-11-22 Thread Reid Kleckner via cfe-commits
rnk abandoned this revision. rnk added a comment. Superseded by https://reviews.llvm.org/D26984 https://reviews.llvm.org/D26944 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D26944: Suppress -Wliblto warning when -flto is not present

2016-11-21 Thread Mehdi Amini via cfe-commits
FYI, doesn’t look good to me, cf what I described on the other thread. > On Nov 21, 2016, at 9:51 PM, Nico Weber wrote: > > Lgtm, works for me. Seems like a good compromise. > > > On Nov 21, 2016 6:29 PM, "Reid Kleckner" > wrote: > rnk created this revision. > rnk adde

Re: [PATCH] D26944: Suppress -Wliblto warning when -flto is not present

2016-11-21 Thread Nico Weber via cfe-commits
Lgtm, works for me. Seems like a good compromise. On Nov 21, 2016 6:29 PM, "Reid Kleckner" wrote: > rnk created this revision. > rnk added reviewers: mehdi_amini, thakis. > rnk added a subscriber: cfe-commits. > > Without -flto, it's unlikely that users will experience LLVM version > skew. The s

[PATCH] D26944: Suppress -Wliblto warning when -flto is not present

2016-11-21 Thread Reid Kleckner via cfe-commits
rnk created this revision. rnk added reviewers: mehdi_amini, thakis. rnk added a subscriber: cfe-commits. Without -flto, it's unlikely that users will experience LLVM version skew. The system installation of libLTO should be able to handle any pre-compiled LLVM bitcode from static archives. http