- User Since
- Mar 16 2015, 3:52 PM (239 w, 3 d)
Mon, Oct 7
Sep 11 2019
- Do not have a declaration as a prevailing symbol in llvm-lto2's arguments
- Fix comments in test
@tejohnson Did my reply about shared metadata clarify your question?
Sep 9 2019
Check for OMPRT_Unknown instead of a default case.
Sep 6 2019
Test -static, -static-openmp interaction. Added these only for iomp5 to avoid test-case explosion.
I'll update this review addressing @joerg's reply to cfe-commits:
Needs testing for the -static interaction?
Mention NDK issue https://github.com/android-ndk/ndk/issues/1028.
Sep 5 2019
Does this regress from existing behavior for unused arguments? Currently, -static-libstdc++ -nostdlib issues an unused argument warning for -static-libstdc++, while AFAICT -static=c++stdlib -nostdlib doesn't.
Change parameter name.
Supported this flag for NetBSD and FreeBSD as well.
Sep 4 2019
Update/clean-up comments in the test.
Aug 28 2019
Aug 27 2019
Fix typo reffers -> refers
Aug 8 2019
Aug 6 2019
Avoid extra variable
Thanks Peter. I'll wait for another day for further comments, and if none, will submit this tomorrow morning (pacific time).
Aug 5 2019
Aug 1 2019
Thanks for the workaround @peter.smith. I suggested it to the NDK developer as well. The immediate need for this change is gone, but I agree that this is useful for the various reasons you outlined.
Jul 31 2019
Jul 25 2019
Call strlen(gcov_prefix) only if its not null.
Remove stray new line.
Refactored the loop to call getenv just once.
LGTM for Android.
Jul 24 2019
Jul 1 2019
Jun 25 2019
Skip version script for all MinGW builds, not just when using lld.
Jun 24 2019
Mar 27 2019
Feb 20 2019
Feb 13 2019
Feb 12 2019
Jan 11 2019
Jan 10 2019
Dec 18 2018
Dec 3 2018
Nov 29 2018
Nov 15 2018
Someone just pointed out to me that even in Android P , libm's sincos() just called sin() followed by cos() so this optimization won't be beneficial. It was changed in March 2018  but is not a part of any release yet.
Nov 8 2018
Thanks for the review Teresa. I'm running the tests again after a rebase and will commit this after the tests pass.
Fix a few more comments.
Update based on review comments. Hopefully I did the WeakForLinker->Prevailing
name changes in all the right places.
Nov 7 2018
Nov 6 2018
Rename thinLTOResolveWeakForLinker* to thinLTOResolvePrevailing*
Nov 5 2018
Fix commit message.
Oct 29 2018
Adding reviewers suggested by 'arc cover'.
Sep 21 2018
This file is not built for WIN32.
Sep 14 2018
Sep 12 2018
As I mentioned in the discussion, we decided to carry build rules for the proposed library in downstream. I've updated this to make it more general, and will leave it open in case there's more interest to revive it in the future.
Add empty source file to silence CMake warning.
Support more platforms, similar to libLLVM.so