- User Since
- Aug 15 2016, 6:00 AM (79 w, 3 d)
Mon, Feb 19
Ping for the third time.
Mon, Jan 29
Jan 20 2018
Jan 19 2018
Disclaimer: I have never seen a plugin for LLDB. I've just noticed the wrong path in strace output and fixed it ;-).
Jan 18 2018
Jan 9 2018
Jan 7 2018
Updated the patch to use LLVM_ENABLE_ZLIB directly.
I was talking of:
Jan 6 2018
Well, it's actually more complex. In Gentoo we want to let the user choose whether he wants to build against zlib or without zlib support. So in the end there are two uses to be considered:
Maybe we could. I presumed people are using the separate variables for some reason. We'd have to check that it's properly sanitized when building in-tree and given to cmake though.
Jan 4 2018
Dec 13 2017
Dec 11 2017
Nov 30 2017
Nov 27 2017
Updated to fix unittest failure.
Nov 25 2017
Nov 18 2017
Or even more abstract (if we assume we don't have to check for presence of llvm_tools_dir):
To be honest, as I said before, I'm entirely confused by the logic there, with all the prepending, appending and shuffling around. But if you believe it gives the correct result, I'm all for it.
Nov 17 2017
I don't see how this could break anything, and it's a pretty straightforward change.
Nov 16 2017
Please revert this commit. You've just broken all the stand-alone builds of clang.
Nov 15 2017
I'm worried that having a different first component of VERSION and SOVERSION could confuse some tools like ldconfig. I'm going to see if we can solve it via different install prefixes with RPATHs first.
Nov 14 2017
Hmm, that's an interesting option too. I was thinking of using LLVM_VERSION_SUFFIX originally but it obviously did not go to SOVERSION. Can this have any unforeseen consequences? Should VERSION become *major+suffix-0-minor* then?
We = Gentoo. What for is already described = to install libstdc++ and libc++ versions of LLVM simultaneously for rebuild period. How long = for the time being, or until we find a better working solution.
Nov 11 2017
Yay, I've found a way to get the desired linkage! ;-)
Nov 10 2017
I've rewritten the tests to use unittest, so this patch needs to be updated now. However, you may want to wait a while since @compnerd promised to look at it and he might have a way to get uniqueExternal() linkage.
Thanks. I'll update it to incorporate the lately committed test fixes, and push it later today.
Nov 9 2017
Let's try some more reviewers.
Nov 8 2017
I see that you haven't more luck than I did finding a better solution. How about the code completion failure? I suppose I still have to figure that one out on my own ;-).
Duplicate of D39161.
@frutiger, do you maybe happen to have fixes for the two other test failures? Maybe I'm tracking them down unnecessarily ;-).
Nov 7 2017
====================================================================== FAIL: tests.cindex.test_code_completion.test_code_complete_availability ---------------------------------------------------------------------- Traceback (most recent call last): File "/usr/lib64/python2.7/site-packages/nose/case.py", line 197, in runTest self.test(*self.arg) File "/usr/src/llvm/tools/clang/bindings/python/tests/cindex/test_code_completion.py", line 75, in test_code_complete_availability check_completion_results(cr, expected) File "/usr/src/llvm/tools/clang/bindings/python/tests/cindex/test_code_completion.py", line 10, in check_completion_results assert c in completions AssertionError
The diffs aren't very useful since I had to add a class for each test unit and so everything needed reindenting. You can take my word that changes boil down to:
Nov 2 2017
Nov 1 2017
Oct 31 2017
Oct 25 2017
Oct 17 2017
Oct 12 2017
Ok, here's another idea. Since the objects have been compiled already (and they explicitly depend on gtest via their own deps), let's just remove the extraneous gtest dep.
Ok, I think I've found a better solution. Will start submitting patches soonish.
Oct 2 2017
Yes, I did apply it both to the installed CMake files and the external LLVM source tree used by compiler-rt. I can't see where else it could have gotten an old version from, and I don't grep any AddLLVM includes either.
Well, I think I did everything right, yet I still get:
Could you give me a few minutes to test it before merging, please?