[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-10-30 Thread Michał Górny via Phabricator via cfe-commits
mgorny updated this revision to Diff 171646. mgorny added a comment. Rebased. https://reviews.llvm.org/D47817 Files: lib/sanitizer_common/CMakeLists.txt lib/sanitizer_common/sanitizer_platform_limits_posix.cc Index: lib/sanitizer_common/sanitizer_platform_limits_posix.cc

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-07-09 Thread Michał Górny via Phabricator via cfe-commits
mgorny added a comment. In https://reviews.llvm.org/D47817#1155788, @Lekensteyn wrote: > In https://reviews.llvm.org/D47817#1155717, @mgorny wrote: > > > > This would be the first user of pkg-config here. I am not sure if this > > > would be the best fix. Usually you cannot (easily) recompile

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-07-09 Thread Peter Wu via Phabricator via cfe-commits
Lekensteyn added a comment. In https://reviews.llvm.org/D47817#1155717, @mgorny wrote: > > This would be the first user of pkg-config here. I am not sure if this > > would be the best fix. Usually you cannot (easily) recompile libc and > > override it, but for external libs such as libtirpc

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-07-09 Thread Michał Górny via Phabricator via cfe-commits
mgorny added a comment. In https://reviews.llvm.org/D47817#1155637, @Lekensteyn wrote: > Hi, thank you for the patch. First a disclaimer, I am not familiar with this > RPC API at all. Me neither. I'm only familiar with this particular issue because it's what we're hitting a lot recently. >

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-07-09 Thread Peter Wu via Phabricator via cfe-commits
Lekensteyn added a comment. Hi, thank you for the patch. First a disclaimer, I am not familiar with this RPC API at all. This would be the first user of pkg-config here. I am not sure if this would be the best fix. Usually you cannot (easily) recompile libc and override it, but for external

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-06-15 Thread Michał Górny via Phabricator via cfe-commits
mgorny added a comment. Gentle ping. Repository: rCRT Compiler Runtime https://reviews.llvm.org/D47817 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D47817: [sanitizer_common] Fix using libtirpc on Linux

2018-06-06 Thread Michał Górny via Phabricator via cfe-commits
mgorny created this revision. mgorny added reviewers: ygribov, kcc. mgorny added a project: Sanitizers. Herald added subscribers: Sanitizers, llvm-commits, kubamracek. mgorny added a reviewer: samsonov. Fix using libtirpc on Linux by using pkg-config to detect it, and append appropriate include