User Details
- User Since
- Jan 20 2015, 10:23 AM (326 w, 1 d)
May 26 2015
- Comment constant
- Use nullptr
- Get modification time for adb push from source file (Pass FileSpec directly through)
- Hard-code file mode flags (to not rely on platform definitions)
- Return failure if a failure occurs reading the source file.
May 22 2015
May 21 2015
May 20 2015
Make static and use function indentation conventions.
May 15 2015
And done in one more location that slipped by before :-).
Use platformIsDarwin instead of manually checking if the platform is in darwin triples.
May 14 2015
zturner, I added a comment about Windows like you suggested.
- Check for hexagon machine.
- Added comment to fix object arch on Windows and detect global / anonymous namespace symbols there.
Check for compatible architecture as well as platform.
Use + to concatenate the platform lists.
May 13 2015
May 12 2015
Removed class and restored to comparison to index, and ensured that the returned arch is linux, otherwise return no supported archs.
May 11 2015
Repeating my comment in what seems to be the active patch. Do we need to enumerate all of the architectures? I put up http://reviews.llvm.org/D9672 earlier today to get non x86 working again where it queries the remote PlatformGdbServer if not the host. WDTY?
Hey, do we need to enumerate all of the architectures? I put up http://reviews.llvm.org/D9672 to get non x86 working again where it queries the remote PlatformGdbServer if not the host. WDTY?
Thanks for your help diagnosing this, I've put up http://reviews.llvm.org/D9672 to hopefully fix the supported architecture issues.
May 9 2015
May 8 2015
lgtm, thanks
May 7 2015
- Delegate to Platform to convert Mmap flags.
- Fix platform when running remote-linux from mac by hard-coding linux archs, previously used host binary arch spec (e.g. macosx when running from mac)
May 5 2015
Looking up the mmap function address from the symbol table seems to have started working - we should still load debug symbols when possible but that can be done in a separate change.
May 1 2015
As a sidenote, I wonder how many of these skip if linux checks should also be skipping if freebsd.
Apr 27 2015
I'm a little confused about why you're setting EXECUTABLE_SUFFIX in Android.cmake but checking CMAKE_EXECUTABLE_SUFFIX in Makefile.rules, shouldn't those both be CMAKE_EXECUTABLE_SUFFIX?
Apr 24 2015
The lovely test suite caught a couple errors which I've addressed in this patch
- NULL is acceptable input for arch or uuid to Symbols::FindSymbolFileInBundle to ignore that attribute.
- Properly check for dsym symbols even if there's no symbol filename in the module spec.
Apr 23 2015
Greg, please check if I'm checking the module spec correctly. I'm not sure if I should only be looking for matching UUID and arch whereas the ModuleSpecList::FindMatchingModuleSpec seems like it will check the FileSpec and PlatformFileSpec as well. It passes the stripped symbols test though which I presume is testing it.
Use ObjectFile::GetModuleSpecifications to check for matching module spec.
I think we need to do the same in test/make/Makefile.rules for AR and OBJCOPY.
Apr 22 2015
Apr 21 2015
I wanted to float this patch by you to get an early impression. Previously we only looked for dsym debugging info on mac and only .debug on linux. This patch should allow us to locate either. What I've tried to do is move as much of the symbol location into common/Symbols.cpp that I can. What do you think of this direction?
Apr 20 2015
Apr 17 2015
Use consistent immediate expansion of CROSS_COMPILE in AR and OBJCOPY.
Apr 16 2015
LGTM, this gets at least 3 more tests passing when running maxosx -> linux, woohoo!
TestCPPExceptionBreakpoint.py
TestCPPExceptionBreakpoints.py
TestGlobalVariables.py
Remove platformIsLinux rather than start creating platformIsX for each platform string getPlatform can return.
Apr 15 2015
Revert first attempt to escape environment variables - need more platform specific investigation.
Apr 13 2015
I'll try to track down which commit(s) broke the tests by applying this patch to earlier revisions.
Apr 10 2015
This patch effectively re-enabled all of the following tests on darwin:
TestAddDsymCommand.py TestAddDsymMidExecutionCommand.py TestConstStrings.py TestFoundationDisassembly.py TestObjCBreakpoints.py TestObjCDynamicSBType.py TestObjCMethodReturningBOOL.py TestObjCMethods2.py TestObjCMethods.py TestObjcOptimized.py TestPrintObj.py TestRdar10967107.py TestRdar11355592.py TestRdar12408181.py TestRuntimeTypes.py TestSetData.py TestSymbolTable.py
Escape quotes and backslashes in environment variable assignments.
Apr 9 2015
This seems to have broken the xcode build for me - confirmed that reverting fixes the build failure:
Apr 8 2015
Looks like the skipUnlessPlatform decorator works with individual test cases, but not entire classes. I believe I have a fix in http://reviews.llvm.org/D8903.