Page MenuHomePhabricator

srhines (Stephen Hines)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 23 2012, 7:46 PM (342 w, 4 d)

Recent Activity

Wed, Mar 13

srhines updated subscribers of D59264: [Driver] Support compiler-rt crtbegin.o/crtend.o for Linux.
Wed, Mar 13, 5:02 PM · Restricted Project

Thu, Mar 7

srhines updated subscribers of D59109: Add --unwindlib=[libgcc|compiler-rt] to parallel --rtlib= [take 2].
Thu, Mar 7, 3:11 PM · Restricted Project

Wed, Feb 20

srhines accepted D58477: [Driver] Fix float ABI default for Android ARMv8..

Dan, this seems pretty important for the NDK. If you submit this, would you want it cherry-picked ASAP?

Wed, Feb 20, 6:03 PM · Restricted Project, Restricted Project

Feb 15 2019

srhines added a comment to D58266: [MC] Sort DWARF FDEs by the associated CIE before emitting them..

I asked Chris to comment here about libunwindstack. Hopefully we can get the problem fixed, but I think this workaround might be necessary for code running on older devices in any case (i.e. NDK apps).

Feb 15 2019, 10:02 AM · Restricted Project

Feb 14 2019

srhines added a comment to D58248: [EarlyCSE & MSSA] Cap the clobbering calls in EarlyCSE..

Very cool, thank you for coming up with a great way to improve this case.

Feb 14 2019, 1:06 PM · Restricted Project
srhines updated subscribers of D58248: [EarlyCSE & MSSA] Cap the clobbering calls in EarlyCSE..
Feb 14 2019, 1:01 PM · Restricted Project

Feb 7 2019

srhines added a comment to D57930: [Driver] Verify GCCInstallation is valid.

Would it be reasonable to have a test for this with perhaps an invalid GCC installation? There is some mock GCC/sysroot testing in https://github.com/llvm/llvm-project/blob/master/clang/test/Driver/android-gcc-toolchain.c and https://github.com/llvm/llvm-project/blob/master/clang/test/Driver/android-ndk-standalone.cpp. I am not sure that it will be easy to trip this same bug that way, but I think it is possible.

Feb 7 2019, 5:09 PM · Restricted Project
srhines accepted D57911: [LLDB] Remove all abandoned LLDB bots.

LGTM for removing the broken Android builders.

Feb 7 2019, 3:46 PM

Feb 6 2019

srhines committed rC353318: Switch to cantFail(), since it does the same assertion..
Switch to cantFail(), since it does the same assertion.
Feb 6 2019, 10:00 AM
srhines committed rGe0a89091d663: Switch to cantFail(), since it does the same assertion. (authored by srhines).
Switch to cantFail(), since it does the same assertion.
Feb 6 2019, 10:00 AM
srhines committed rL353318: Switch to cantFail(), since it does the same assertion..
Switch to cantFail(), since it does the same assertion.
Feb 6 2019, 9:59 AM
srhines closed D36806: Switch to cantFail(), since it does the same assertion..
Feb 6 2019, 9:59 AM · Restricted Project

Feb 5 2019

srhines added a comment to D36806: Switch to cantFail(), since it does the same assertion..

Looks like this was LGTM'd but never applied. Stephen -- do you have commit access?

Feb 5 2019, 2:10 PM · Restricted Project

Jan 30 2019

srhines added a reviewer for D57143: [builtins] Rounding mode support for addxf3/subxf3: peter.smith.
Jan 30 2019, 9:08 AM · Restricted Project, Restricted Project

Jan 22 2019

srhines accepted D56977: [builtins] Do not set hidden attribute on Android.
Jan 22 2019, 4:47 PM

Jan 8 2019

srhines accepted D56456: [Driver] Default to -fno-addrsig on Android..
Jan 8 2019, 2:28 PM
srhines accepted D55953: Android is not GNU, so don't claim that it is..

Sorry about the delay.

Jan 8 2019, 1:36 PM

Dec 11 2018

srhines accepted D55587: [hwasan] Verify Android TLS slot at startup..
Dec 11 2018, 4:56 PM

Dec 7 2018

srhines added inline comments to D34796: upporting -f(no)-reorder-functions flag, clang side change.
Dec 7 2018, 10:15 AM

Dec 3 2018

srhines added a comment to D53850: Declares __cpu_model as dso local.

Craig, does this look ok now?

Dec 3 2018, 5:27 PM

Nov 30 2018

srhines updated subscribers of D54125: [LTO] Drop non-prevailing definitions only if linkage is not local or appending.
Nov 30 2018, 11:48 AM

Nov 28 2018

srhines added a comment to D55029: set default max-page-size to 4KB in lld for Android Aarch64.

I wouldn't rush to submit this now, given that this issue is not new at all. Maybe we can just wait for Peter's response?

Nov 28 2018, 3:09 PM
srhines accepted D55029: set default max-page-size to 4KB in lld for Android Aarch64.

Thanks Zhizhou! I'm curious about Peter's answer to the 64KiB default as well, but I think we should move forward with getting this patch submitted.

Nov 28 2018, 2:52 PM

Nov 16 2018

srhines added a comment to D52025: [TargetLowering] Android has sincos functions.

I just wanted to follow up here.

Nov 16 2018, 7:51 PM

Nov 1 2018

srhines added a comment to D53906: [ARM][AArch64] Increase TLS alignment to reserve space for Android's TCB.

Yabin did some of the work to bring TSan to Android. Perhaps he knows whether this is has significant use. Considering that it is meant as a debugging aide, I would actually be comfortable with moving it. You are also correct that we never officially marked it as a feature of the NDK (or platform). That is something we want to do though.

Nov 1 2018, 5:25 PM
srhines updated subscribers of D53906: [ARM][AArch64] Increase TLS alignment to reserve space for Android's TCB.
Nov 1 2018, 5:23 PM

Oct 26 2018

srhines updated subscribers of D53765: [RFC prototype] Implementation of asm-goto support in LLVM.
Oct 26 2018, 11:24 PM · Restricted Project

Oct 25 2018

srhines added a comment to D53742: [XRay] Use std::errc::invalid_argument instead of std::errc::bad_message.

LGTM, but please wait for dberris approval.

Oct 25 2018, 9:41 PM

Oct 24 2018

srhines updated subscribers of D53486: [libcxx] Only define __libcpp_is_floating_point<_Float16> for Clang.
Oct 24 2018, 9:19 AM

Oct 22 2018

srhines accepted D53529: [Driver] fix broken test.
Oct 22 2018, 2:25 PM

Oct 19 2018

srhines accepted D53463: [Driver] allow Android triples to alias for non Android targets.

Please remove the reference to b/X in the commit message. LLVM doesn't allow internal bug numbers, and you described the issue well enough without it.

Oct 19 2018, 6:14 PM

Oct 10 2018

srhines accepted D53118: [Driver] Fix --hash-style choice for Android..
Oct 10 2018, 6:28 PM
srhines accepted D53117: [Driver] Default to `-z now` and `-z relro` on Android..
Oct 10 2018, 6:25 PM
srhines added a comment to D53121: [Driver] Add defaults for Android ARM FPUs..

This LGTM, but we should wait to hear from Kristof before submitting.

Oct 10 2018, 6:23 PM
srhines added a reviewer for D53121: [Driver] Add defaults for Android ARM FPUs.: kristof.beyls.
Oct 10 2018, 6:21 PM
srhines accepted D53109: [Driver] Default Android toolchains to libc++..

Really cool! Thanks for making everything easier to use out-of-the-box.

Oct 10 2018, 6:19 PM

Oct 8 2018

srhines added a reviewer for D53003: [ELF] Fix link failure with Android compressed relocation support.: rprichard.
Oct 8 2018, 4:04 PM

Oct 2 2018

srhines committed rCRT343599: [sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan.
[sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan
Oct 2 2018, 9:22 AM
srhines committed rL343599: [sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan.
[sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan
Oct 2 2018, 9:22 AM
srhines closed D52770: [sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan.
Oct 2 2018, 9:21 AM
srhines created D52770: [sanitizer] Use -Wl,-z,global on Android for sanitizers except UBsan.
Oct 2 2018, 12:17 AM

Sep 20 2018

srhines added inline comments to D52248: [SEMA] ignore duplicate declaration specifiers from typeof exprs.
Sep 20 2018, 1:48 PM

Sep 19 2018

srhines accepted D52251: [builtins] Add __emutls_unregister_key function.

+enh and danalbert - in case they have any other concerns about this (Ryan's already on the review). From my perspective, everything here makes sense for resolving this issue.

Sep 19 2018, 3:13 PM
srhines updated subscribers of D52251: [builtins] Add __emutls_unregister_key function.
Sep 19 2018, 3:09 PM

Sep 18 2018

srhines added a comment to D52191: Fix logic around determining use of frame pointer with -pg..

Thanks @dblaikie for the quick fixup. I must have accidentally dropped the '!', because I did run check-all to test the change.

Sep 18 2018, 1:21 PM
srhines committed rC342501: Fix logic around determining use of frame pointer with -pg..
Fix logic around determining use of frame pointer with -pg.
Sep 18 2018, 11:38 AM
srhines committed rL342501: Fix logic around determining use of frame pointer with -pg..
Fix logic around determining use of frame pointer with -pg.
Sep 18 2018, 11:38 AM
srhines closed D52191: Fix logic around determining use of frame pointer with -pg..
Sep 18 2018, 11:38 AM
srhines closed D52191: Fix logic around determining use of frame pointer with -pg..
Sep 18 2018, 11:38 AM
srhines added a comment to D52191: Fix logic around determining use of frame pointer with -pg..

Sure, looks good. Though my other/vague concern is why does this case error about fomit-frame-pointer having no effect, but other things (like using -fomit-frame-pointer on a target that requires frame pointers) that ignore -fomit-frame-pointer don't? Weird. But it probably makes sense somehow.

Sep 18 2018, 11:34 AM

Sep 17 2018

srhines created D52191: Fix logic around determining use of frame pointer with -pg..
Sep 17 2018, 1:58 PM
srhines added a comment to D50297: Align AArch64 and i386 image base to superpage.

+Ryan in case there is anything here that could affect Bionic loading from these pages.

Sep 17 2018, 10:53 AM
srhines added a reviewer for D50297: Align AArch64 and i386 image base to superpage: rprichard.
Sep 17 2018, 10:49 AM
srhines accepted D52163: -S as an alias for --strip-all-gnu.

Thanks again! @alexshap feel free to submit this or I can do it in the morning tomorrow (can't stay up any later to watch for unlikely breakage, etc.).

Sep 17 2018, 2:25 AM
srhines added inline comments to D52163: -S as an alias for --strip-all-gnu.
Sep 17 2018, 1:59 AM
srhines added a comment to D52163: -S as an alias for --strip-all-gnu.

Thanks for noticing this missing feature, as well as for the patch! You really should add a simple test to test/tools/llvm-objcopy/strip-all-gnu.test so that this doesn't ever regress. The test strip-all-and-keep-symbol.test is an example that has multiple RUN lines. Here is what you need to change the strip-all-gnu.test to (I believe) in order to get this working:

Sep 17 2018, 1:35 AM

Sep 13 2018

srhines committed rL342165: Support -fno-omit-frame-pointer with -pg..
Support -fno-omit-frame-pointer with -pg.
Sep 13 2018, 12:51 PM
srhines committed rC342165: Support -fno-omit-frame-pointer with -pg..
Support -fno-omit-frame-pointer with -pg.
Sep 13 2018, 12:51 PM
srhines closed D51713: Support -fno-omit-frame-pointer with -pg..
Sep 13 2018, 12:51 PM
srhines added inline comments to D18086: Fix default processor name for armv6k..
Sep 13 2018, 12:33 PM

Sep 12 2018

srhines added a comment to D51713: Support -fno-omit-frame-pointer with -pg..

What is the call generated with -pg for AMR32, gnu_mcount_nc or _mount? gnu_mcount_nc with "-pg" is known to be broken ( https://bugs.llvm.org/show_bug.cgi?id=33845)

Sep 12 2018, 2:41 PM

Sep 7 2018

srhines accepted D51693: ADT: add <bit> header, implement C++20 bit_cast, use.

Union punning is one of my pet peeves, so I am glad to see this. bit_cast seems generally useful too, so thanks for adding it.

Sep 7 2018, 11:33 AM

Sep 5 2018

srhines added a comment to D51671: [ELF] Set Out::TlsPhdr earlier for encoding packed reloc tables.

Thanks for the update. I'll let the project maintainers give a final LGTM, but I'm happy with it.

Sep 5 2018, 7:55 PM
srhines added a comment to D51713: Support -fno-omit-frame-pointer with -pg..

http://b/32510864 was the internal bug request, so I am noting it here for future reference, but I think that the patch itself is pretty self-explanatory (rather than filing a distinct upstream bug about this issue).

Sep 5 2018, 7:40 PM
srhines added a comment to D51713: Support -fno-omit-frame-pointer with -pg..

This was discovered in the Android build system (which passes -fomit-frame-pointer by default for ARM configurations. This leads to the inability to specify -pg, since there is no way to override the mere presence of -fomit-frame-pointer.

Sep 5 2018, 7:39 PM
srhines updated subscribers of D51713: Support -fno-omit-frame-pointer with -pg..
Sep 5 2018, 7:37 PM
srhines created D51713: Support -fno-omit-frame-pointer with -pg..
Sep 5 2018, 7:37 PM
srhines added inline comments to D51671: [ELF] Set Out::TlsPhdr earlier for encoding packed reloc tables.
Sep 5 2018, 2:35 AM
srhines added a comment to D51671: [ELF] Set Out::TlsPhdr earlier for encoding packed reloc tables.

Thanks for the detailed explanations. I only had one minor nitpick.

Sep 5 2018, 1:04 AM

Aug 31 2018

srhines added inline comments to D51538: Add glibc_prereq to platform limits mmsghdr.
Aug 31 2018, 3:23 PM
srhines added inline comments to D51538: Add glibc_prereq to platform limits mmsghdr.
Aug 31 2018, 3:05 PM
srhines added a comment to D51538: Add glibc_prereq to platform limits mmsghdr.

Please upload patches with context, as it makes it easier to review. https://llvm.org/docs/Phabricator.html has instructions on how to do this (search for "context").

Aug 31 2018, 10:37 AM
srhines accepted D51521: Refactor Addlibgcc to make the when and what logic more straightfoward..

Thanks for cleaning this up and adding better checks for Android. :)

Aug 31 2018, 9:34 AM

Aug 30 2018

srhines added inline comments to D51502: [X86] Fix register resizings for inline assembly register operands..
Aug 30 2018, 1:11 PM

Aug 28 2018

srhines accepted D45588: Start reserving x18 by default on Android targets..

Sorry, this got buried in my inbox back in April right before EuroLLVM.

Aug 28 2018, 6:00 PM

Aug 21 2018

srhines accepted D51068: [Android] Default to -fno-math-errno.
Aug 21 2018, 5:28 PM
srhines added inline comments to D51002: [Tooling] Allow -flto flags and filter out -Wa, flags.
Aug 21 2018, 3:12 PM

Jul 26 2018

srhines committed rL338062: Handle the lack of a symbol table correctly..
Handle the lack of a symbol table correctly.
Jul 26 2018, 1:06 PM
srhines closed D49534: Handle the lack of a symbol table correctly..
Jul 26 2018, 1:05 PM
srhines added a comment to D49534: Handle the lack of a symbol table correctly..

@srhines - is there anything blocking this diff ?

Jul 26 2018, 1:01 PM

Jul 24 2018

srhines added a comment to D49751: Add maybe-unused attribute to a variable..

Definitely deal with the submodule stuff. We don't want a new file checked in for tools/clang.

Jul 24 2018, 1:19 PM

Jul 20 2018

srhines added a comment to D49534: Handle the lack of a symbol table correctly..

I restored the original change, since I had confused things with arcanist before. PTAL

Jul 20 2018, 3:50 PM
srhines updated the diff for D49534: Handle the lack of a symbol table correctly..

Revert back to not layering the positional argument change.

Jul 20 2018, 3:48 PM

Jul 19 2018

srhines abandoned D49537: Fix positional output argument for llvm-strip..

Gah, you are correct. Our Android build system had grown to depend on this behavior from llvm-strip previously. Doing an update now is how I am finding all of these issues. I'll fix it in our build system and abandon this CL. Thanks for the review.

Jul 19 2018, 12:38 PM
srhines updated the diff for D49537: Fix positional output argument for llvm-strip..

Switch to using .back() because I was lazy in the first upload. :)

Jul 19 2018, 4:30 AM
srhines created D49537: Fix positional output argument for llvm-strip..
Jul 19 2018, 4:17 AM
srhines updated the diff for D49534: Handle the lack of a symbol table correctly..
  • Fix positional output argument for llvm-strip.
Jul 19 2018, 4:06 AM
srhines updated the diff for D49534: Handle the lack of a symbol table correctly..

Ran git-clang-format and added rationale to the test.

Jul 19 2018, 1:55 AM
srhines created D49534: Handle the lack of a symbol table correctly..
Jul 19 2018, 1:51 AM

Jul 12 2018

srhines committed rL336921: Add --strip-all option back to llvm-strip..
Add --strip-all option back to llvm-strip.
Jul 12 2018, 10:47 AM
srhines closed D49226: Add --strip-all option back to llvm-strip..
Jul 12 2018, 10:47 AM
srhines created D49226: Add --strip-all option back to llvm-strip..
Jul 12 2018, 2:56 AM

Jul 10 2018

srhines committed rCRT336749: Add libcxxabi option back for sanitizer use..
Add libcxxabi option back for sanitizer use.
Jul 10 2018, 5:55 PM
srhines committed rL336749: Add libcxxabi option back for sanitizer use..
Add libcxxabi option back for sanitizer use.
Jul 10 2018, 5:55 PM
srhines closed D49157: Add libcxxabi option back for sanitizer use..
Jul 10 2018, 5:55 PM
srhines added a comment to D49157: Add libcxxabi option back for sanitizer use..

Thanks for the quick review.

Jul 10 2018, 5:47 PM
srhines added a comment to D49157: Add libcxxabi option back for sanitizer use..

https://reviews.llvm.org/D47100 is the commit where this got dropped.

Jul 10 2018, 3:07 PM
srhines updated subscribers of D49157: Add libcxxabi option back for sanitizer use..
Jul 10 2018, 3:06 PM
srhines created D49157: Add libcxxabi option back for sanitizer use..
Jul 10 2018, 3:05 PM

Jul 6 2018

srhines abandoned D48459: Respect CMAKE_SYSROOT and CMAKE_CROSSCOMPILING when searching for libxml2..

Removed this in favor of the suggestions here. Setting the CMAKE_FIND_ROOT_PATH_MODE* variables does make this properly hermetic.

Jul 6 2018, 8:58 PM