Page MenuHomePhabricator

friss (Frederic Riss)
User

Projects

User does not belong to any projects.

User Details

User Since
Sep 4 2014, 1:00 AM (253 w, 6 d)

Recent Activity

Thu, Jul 11

friss added a comment to D64546: [lldb] Make TestDeletedExecutable more reliable.

This might mitigate the issue, but timeouts like this are bound to fail in some circumstances (machine load, ...). It's more work, but can we instead have the inferior produce an observable side effect (eg, print some output) and synchronize on this?

Thu, Jul 11, 7:23 AM · Restricted Project, Restricted Project

Tue, Jun 25

friss added a comment to D63745: [CMake] Check that a certificate for lldb is present at build time..

On second thought, let's check that LLDB_CODESIGN_IDENTITY equals lldb_codesign before doing this check.

This question isn't important but I'm kind of curious: Does it have to be called lldb_codesign? Could you have an arbitrary identity and then sign with that, assuming the cert exists, or does debugserver expect a cert with that name exactly?

Tue, Jun 25, 12:17 AM · Restricted Project, Restricted Project

May 28 2019

friss added inline comments to D62472: [CMake] LLDB.framework tools handling.
May 28 2019, 11:04 AM · Restricted Project, Restricted Project

May 22 2019

friss committed rGbb2b52769b40: Actaully lock accesses to OptionValueFileSpecList objects (authored by friss).
Actaully lock accesses to OptionValueFileSpecList objects
May 22 2019, 2:57 PM
friss committed rL361440: Actaully lock accesses to OptionValueFileSpecList objects.
Actaully lock accesses to OptionValueFileSpecList objects
May 22 2019, 2:57 PM
friss committed rLLDB361440: Actaully lock accesses to OptionValueFileSpecList objects.
Actaully lock accesses to OptionValueFileSpecList objects
May 22 2019, 2:57 PM

May 10 2019

friss accepted D61759: Switch to FindSymbolsMatchingRegExAndType() from FindFunctions() in FindLibCppStdFunctionCallableInfo().

This seems obviously better, go for it.

May 10 2019, 4:24 PM · Restricted Project
friss added a comment to D61805: Add nullptr check in FindLibCppStdFunctionCallableInfo().

Do you have a testcase triggering this?

May 10 2019, 4:24 PM

May 1 2019

friss accepted D61146: Set a CXXRecordDecl to not be passed in registers if DW_CC_pass_by_reference when loading from DWARF.

Thanks, indeed this looks better.

May 1 2019, 2:33 PM · Restricted Project
friss added inline comments to D61146: Set a CXXRecordDecl to not be passed in registers if DW_CC_pass_by_reference when loading from DWARF.
May 1 2019, 1:59 PM · Restricted Project
friss added inline comments to D61146: Set a CXXRecordDecl to not be passed in registers if DW_CC_pass_by_reference when loading from DWARF.
May 1 2019, 1:07 PM · Restricted Project
friss added a comment to D61146: Set a CXXRecordDecl to not be passed in registers if DW_CC_pass_by_reference when loading from DWARF.

Correct me if I'm wrong, but the test seems overly complicated still. We are testing that we can get the return value of a type that needs to be passed by reference. Calling simple free function Bounds bounds() should trigger this code path, shouldn't it? I would be interested to know wether we do the right thing when passing such a struct by value in the source code too as Clang doesn't seem to differentiate return values and arguments. Something like:

// This structure has a non-trivial copy constructor so
// it needs to be passed by reference.
struct PassByRef {
  PassByRef() = default;
  PassByRef(const PassByRef &p);
May 1 2019, 10:24 AM · Restricted Project

Apr 30 2019

friss accepted D61266: Skip TestClassTemplateParameterPack.py on all platforms.

LGTM if no one else objects.

Apr 30 2019, 9:51 AM

Apr 29 2019

friss added inline comments to D61266: Skip TestClassTemplateParameterPack.py on all platforms.
Apr 29 2019, 2:35 PM
friss requested changes to D61266: Skip TestClassTemplateParameterPack.py on all platforms.

We shouldn't skip the whole test, just the expressions that worked for bad reasons. Calling functions on local variables should work, it's just creating the templated objects that is completely broken.

Apr 29 2019, 12:46 PM

Apr 26 2019

friss committed rGfad0aa1e3a1c: Pass explicit C++ version to test (authored by friss).
Pass explicit C++ version to test
Apr 26 2019, 2:17 PM
friss committed rLLDB359349: Pass explicit C++ version to test.
Pass explicit C++ version to test
Apr 26 2019, 2:14 PM
friss committed rL359349: Pass explicit C++ version to test.
Pass explicit C++ version to test
Apr 26 2019, 2:14 PM
friss committed rGb6661490b42f: TestZMMRegister: use an integer division as intended (authored by friss).
TestZMMRegister: use an integer division as intended
Apr 26 2019, 1:24 PM
friss committed rL359347: TestZMMRegister: use an integer division as intended.
TestZMMRegister: use an integer division as intended
Apr 26 2019, 1:24 PM
friss committed rLLDB359347: TestZMMRegister: use an integer division as intended.
TestZMMRegister: use an integer division as intended
Apr 26 2019, 1:24 PM

Apr 24 2019

friss committed rG2f4967632158: Skip test introduced in r359140 on windows (authored by friss).
Skip test introduced in r359140 on windows
Apr 24 2019, 2:59 PM
friss committed rL359145: Skip test introduced in r359140 on windows.
Skip test introduced in r359140 on windows
Apr 24 2019, 2:59 PM
friss committed rLLDB359145: Skip test introduced in r359140 on windows.
Skip test introduced in r359140 on windows
Apr 24 2019, 2:59 PM
friss committed rG265df39a80da: Fix infinite recursion when calling C++ template functions (authored by friss).
Fix infinite recursion when calling C++ template functions
Apr 24 2019, 2:03 PM
friss committed rL359140: Fix infinite recursion when calling C++ template functions.
Fix infinite recursion when calling C++ template functions
Apr 24 2019, 2:02 PM
friss committed rLLDB359140: Fix infinite recursion when calling C++ template functions.
Fix infinite recursion when calling C++ template functions
Apr 24 2019, 2:02 PM
friss closed D61044: Fix infinite recursion when calling C++ template functions.
Apr 24 2019, 2:02 PM · Restricted Project

Apr 23 2019

friss created D61044: Fix infinite recursion when calling C++ template functions.
Apr 23 2019, 3:48 PM · Restricted Project
friss committed rGacbf0058e93d: Lock accesses to OptionValueFileSpecList objects (authored by friss).
Lock accesses to OptionValueFileSpecList objects
Apr 23 2019, 1:17 PM
friss committed rL359028: Lock accesses to OptionValueFileSpecList objects.
Lock accesses to OptionValueFileSpecList objects
Apr 23 2019, 1:17 PM
friss committed rLLDB359028: Lock accesses to OptionValueFileSpecList objects.
Lock accesses to OptionValueFileSpecList objects
Apr 23 2019, 1:16 PM
friss closed D60468: Lock accesses to OptionValueFileSpecList objects.
Apr 23 2019, 1:16 PM · Restricted Project
friss added a comment to D60468: Lock accesses to OptionValueFileSpecList objects.

There were no objections to the patch, and it fixes a real crash seen in the field so I'm going to check it in.

Apr 23 2019, 11:40 AM · Restricted Project

Apr 18 2019

friss accepted D60862: [CMake] Allow custom extensions for externalized debug info.

Interesting. Cmake scoping rules are still a mystery to me. So when you `set(FOO ...), it's set only for the current scope and in any functions/macros you call in that scope? Then this seems like an ok solution.

Apr 18 2019, 9:12 AM · Restricted Project
friss requested changes to D60862: [CMake] Allow custom extensions for externalized debug info.

Actually, thinking more about this, how do you use it? LLVM_EXTERNALIZE_DEBUGINFO_EXTENSION is a global but this would be different per target.

Apr 18 2019, 8:14 AM · Restricted Project
friss accepted D60862: [CMake] Allow custom extensions for externalized debug info.

LGTM

Apr 18 2019, 7:44 AM · Restricted Project

Apr 17 2019

friss committed rG8ecb7bbe2bab: [CMake] Remove Apple-specific version logic. (authored by friss).
[CMake] Remove Apple-specific version logic.
Apr 17 2019, 11:22 AM
friss committed rLLDB358604: [CMake] Remove Apple-specific version logic..
[CMake] Remove Apple-specific version logic.
Apr 17 2019, 11:22 AM
friss committed rL358604: [CMake] Remove Apple-specific version logic..
[CMake] Remove Apple-specific version logic.
Apr 17 2019, 11:22 AM

Apr 16 2019

friss committed rGb47d3560938d: [debugserver] Relax the codesigning identity check (authored by friss).
[debugserver] Relax the codesigning identity check
Apr 16 2019, 1:56 PM
friss committed rLLDB358525: [debugserver] Relax the codesigning identity check.
[debugserver] Relax the codesigning identity check
Apr 16 2019, 1:53 PM
friss committed rL358525: [debugserver] Relax the codesigning identity check.
[debugserver] Relax the codesigning identity check
Apr 16 2019, 1:53 PM

Apr 15 2019

friss added inline comments to D60642: [dsymutil] Collect parseable Swift interfaces in the .dSYM bundle..
Apr 15 2019, 9:12 AM · Restricted Project

Apr 10 2019

friss added a comment to D59960: Fix for ambiguous lookup in expressions between local variable and namespace.

You didn't address my comment that "this" needs to treated specially in Obj-C++ too. Other than that this LGTM

Apr 10 2019, 5:04 PM · Restricted Project

Apr 9 2019

friss added inline comments to D59960: Fix for ambiguous lookup in expressions between local variable and namespace.
Apr 9 2019, 12:37 PM · Restricted Project
friss added a comment to D60468: Lock accesses to OptionValueFileSpecList objects.

No opinion on the patch, but what is the reason for having settings that are shared between multiple Debugger instances? My expectation was that the debugger objects are completely independent, and I would be surprised if the value of some setting changed from under me because of something that happened in another debug session.

Apr 9 2019, 11:23 AM · Restricted Project
friss added a comment to D60468: Lock accesses to OptionValueFileSpecList objects.

Almost seems like we can build the mutex into the base class OptionValue as we need general threaded protection for every setting. They any function that gets or sets the value should be able to protect itself using the base mutex

Apr 9 2019, 9:35 AM · Restricted Project
friss created D60468: Lock accesses to OptionValueFileSpecList objects.
Apr 9 2019, 9:13 AM · Restricted Project

Apr 3 2019

friss accepted D60178: [Reproducers] Capture return values of functions returning by ptr/ref.

This seems super mechanical and we discussed it at length offline. LGTM

Apr 3 2019, 2:10 PM · Restricted Project, Restricted Project

Mar 28 2019

friss added inline comments to D59960: Fix for ambiguous lookup in expressions between local variable and namespace.
Mar 28 2019, 2:42 PM · Restricted Project

Mar 26 2019

friss added a comment to D59847: Regression test to ensure that we handling importing of std::vector of enums correctly.

As we're just adding test coverage, could we add a little more?

  • Anonymous enum
  • Enum through a typedef
  • class enum
  • enum declared inside of the function rather than at the top-level
  • nested enum in a record type
  • enum nested in a templated class
  • anything else I haven't thought about...
Mar 26 2019, 5:20 PM · Restricted Project

Mar 8 2019

friss committed rG08ae3e0f0b5d: Actually implement the TestQueues.py workaround (authored by friss).
Actually implement the TestQueues.py workaround
Mar 8 2019, 5:35 PM
friss committed rLLDB355766: Actually implement the TestQueues.py workaround.
Actually implement the TestQueues.py workaround
Mar 8 2019, 5:34 PM
friss committed rL355766: Actually implement the TestQueues.py workaround.
Actually implement the TestQueues.py workaround
Mar 8 2019, 5:33 PM
friss committed rG65e062655ed8: Try to workaround the TestQueues.py flakyness (authored by friss).
Try to workaround the TestQueues.py flakyness
Mar 8 2019, 5:23 PM
friss committed rLLDB355764: Try to workaround the TestQueues.py flakyness.
Try to workaround the TestQueues.py flakyness
Mar 8 2019, 5:23 PM
friss committed rL355764: Try to workaround the TestQueues.py flakyness.
Try to workaround the TestQueues.py flakyness
Mar 8 2019, 5:22 PM
friss committed rG7f3c16c0f317: Add more logging to TestQueues.py (authored by friss).
Add more logging to TestQueues.py
Mar 8 2019, 9:11 AM
friss committed rLLDB355706: Add more logging to TestQueues.py.
Add more logging to TestQueues.py
Mar 8 2019, 9:08 AM
friss committed rL355706: Add more logging to TestQueues.py.
Add more logging to TestQueues.py
Mar 8 2019, 9:08 AM

Mar 7 2019

friss committed rG46fac9c4f270: Add logging to TestQueues.py (authored by friss).
Add logging to TestQueues.py
Mar 7 2019, 2:29 PM
friss committed rL355647: Add logging to TestQueues.py.
Add logging to TestQueues.py
Mar 7 2019, 2:29 PM
friss committed rLLDB355647: Add logging to TestQueues.py.
Add logging to TestQueues.py
Mar 7 2019, 2:29 PM
friss committed rGc525b36b434b: Fix TestAppleSimulatorOSType.py with Xcode 10.2 (authored by friss).
Fix TestAppleSimulatorOSType.py with Xcode 10.2
Mar 7 2019, 2:13 PM
friss committed rLLDB355644: Fix TestAppleSimulatorOSType.py with Xcode 10.2.
Fix TestAppleSimulatorOSType.py with Xcode 10.2
Mar 7 2019, 2:13 PM
friss committed rL355644: Fix TestAppleSimulatorOSType.py with Xcode 10.2.
Fix TestAppleSimulatorOSType.py with Xcode 10.2
Mar 7 2019, 2:13 PM

Mar 6 2019

friss committed rGfcda044d22b8: [debugserver] Fix IsUserReady thread filtering (authored by friss).
[debugserver] Fix IsUserReady thread filtering
Mar 6 2019, 1:56 PM
friss committed rL355555: [debugserver] Fix IsUserReady thread filtering.
[debugserver] Fix IsUserReady thread filtering
Mar 6 2019, 1:55 PM
friss committed rLLDB355555: [debugserver] Fix IsUserReady thread filtering.
[debugserver] Fix IsUserReady thread filtering
Mar 6 2019, 1:55 PM
friss closed D58912: [debugserver] Fix IsUserReady thread filtering.
Mar 6 2019, 1:55 PM · Restricted Project

Mar 4 2019

friss added a comment to D58912: [debugserver] Fix IsUserReady thread filtering.

Will this hide a thread that jumps through a null function pointer? That's the only user process case where a pc of 0 needs to be reported to the developer.

Mar 4 2019, 3:48 PM · Restricted Project
friss created D58912: [debugserver] Fix IsUserReady thread filtering.
Mar 4 2019, 10:23 AM · Restricted Project

Feb 27 2019

friss added a comment to D58534: dsymutil support for DW_OP_convert.

LGTM on the surface. I wouldn't mind someone else (@JDevlieghere ?) taking a look.

Feb 27 2019, 1:56 PM · Restricted Project, debug-info

Feb 26 2019

friss added inline comments to D58534: dsymutil support for DW_OP_convert.
Feb 26 2019, 4:09 PM · Restricted Project, debug-info

Feb 14 2019

friss committed rGb3a4649a6a27: [dotest] Fix compiler version number comparison (authored by friss).
[dotest] Fix compiler version number comparison
Feb 14 2019, 10:51 AM
friss committed rL354047: [dotest] Fix compiler version number comparison.
[dotest] Fix compiler version number comparison
Feb 14 2019, 10:51 AM
friss committed rLLDB354047: [dotest] Fix compiler version number comparison.
[dotest] Fix compiler version number comparison
Feb 14 2019, 10:51 AM
friss closed D58219: [dotest] Fix compiler version number comparison.
Feb 14 2019, 10:51 AM · Restricted Project

Feb 13 2019

friss updated the diff for D58219: [dotest] Fix compiler version number comparison.

Use LooseVersion as suggested by Zachary

Feb 13 2019, 5:42 PM · Restricted Project
friss created D58219: [dotest] Fix compiler version number comparison.
Feb 13 2019, 4:49 PM · Restricted Project

Feb 12 2019

friss added inline comments to D57979: [dsymutil] Don't clone empty CUs.
Feb 12 2019, 8:49 AM · Restricted Project

Feb 8 2019

friss added a comment to D57979: [dsymutil] Don't clone empty CUs.

I might be missing something, but empty as in "has no DIEs at all" is invalid. I don't think having one DIE with no children counts as invalid (it might not be really useful, but this review is about it being invalid, right?)

Feb 8 2019, 2:49 PM · Restricted Project

Dec 19 2018

friss added inline comments to D55859: noexternal 2/2: symbols.enable-external-lookup=false on all hosts (not just OSX).
Dec 19 2018, 8:40 AM · Restricted Project
friss added a comment to D55859: noexternal 2/2: symbols.enable-external-lookup=false on all hosts (not just OSX).

Unfortunately, I don't think llvm has anything like that, though I think it would be extremely useful (/me looks at apple folks). If you try hard enough, you should be able to get clang to produce a dsym bundle for you even on linux. This did the trick for me:

$ cat /tmp/a.c 
void start() asm("start");
void dyld_stub_binder() asm("dyld_stub_binder");

void start() {}
void dyld_stub_binder() {}
$ bin/clang --target=x86_64-apple-darwin --debug /tmp/a.c -o /tmp/a.out -fuse-ld=lld -nostdlib
ld64.lld: warning: -sdk_version is required when emitting min version load command.  Setting sdk version to match provided min version
$ ls -l /tmp/a.out.dSYM/Contents/Resources/DWARF
total 12
-rw-rw---- 1 pavel pavel 8852 Dec 19 10:24 a.out

Dec 19 2018, 8:13 AM · Restricted Project

Dec 13 2018

friss added a comment to D55571: [ast] CreateParameterDeclaration should use an appropriate DeclContext..

Zachary, how did you figure out this can be an issue? Does it fix something we should be testing?

Dec 13 2018, 1:50 PM

Dec 10 2018

friss committed rL348805: Reflow a multi-line string and add a newline.
Reflow a multi-line string and add a newline
Dec 10 2018, 2:33 PM
friss committed rLLDB348805: Reflow a multi-line string and add a newline.
Reflow a multi-line string and add a newline
Dec 10 2018, 2:33 PM

Dec 8 2018

friss committed rLLDB348711: Revert a hunk that shouldn't have been included in the last commit..
Revert a hunk that shouldn't have been included in the last commit.
Dec 8 2018, 6:41 PM
friss committed rL348711: Revert a hunk that shouldn't have been included in the last commit..
Revert a hunk that shouldn't have been included in the last commit.
Dec 8 2018, 6:41 PM
friss committed rLLDB348710: Remove some UB in RegisterContextDarwin_arm64.cpp.
Remove some UB in RegisterContextDarwin_arm64.cpp
Dec 8 2018, 6:40 PM
friss committed rL348710: Remove some UB in RegisterContextDarwin_arm64.cpp.
Remove some UB in RegisterContextDarwin_arm64.cpp
Dec 8 2018, 6:39 PM

Dec 5 2018

friss added a comment to D55328: [CMake] Revised LLDB.framework builds.

I like seeing all of the cmake modifications for the LLDB.framework. Are we planning on trying to get rid of the Xcode project at some point soon and use the auto generated one made by cmake?

Dec 5 2018, 9:13 AM
friss added a comment to D55319: [CMake] Proposal: Prefer LLDB_VERSION over plist value in EmbedAppleVersion.cmake.

Wouldn't it make even more sense to to inject LLDB_VERSION into the Info.plist? We will use the Info.plist afterwards, right?

Dec 5 2018, 7:53 AM

Nov 30 2018

friss accepted D54922: [dsymutil] Gather local symbol addresses in addition to global in the main executable..

LGTM

Nov 30 2018, 8:14 AM

Nov 18 2018

friss added a comment to D54680: Don't use lldb -O in lit tests.

I do agree it's slightly easier to read this way. I was conflicted while writing the tests between readability and conciseness. I think this is a good compromise.

Nov 18 2018, 6:23 PM

Nov 16 2018

friss committed rL347117: Make TestAppleSimulatorOSType.py more flexible.
Make TestAppleSimulatorOSType.py more flexible
Nov 16 2018, 4:43 PM
friss committed rLLDB347117: Make TestAppleSimulatorOSType.py more flexible.
Make TestAppleSimulatorOSType.py more flexible
Nov 16 2018, 4:43 PM
friss committed rLLDB347109: Rewrite stop-hook tests as a couple of FileCheck tests.
Rewrite stop-hook tests as a couple of FileCheck tests
Nov 16 2018, 3:10 PM
friss committed rL347109: Rewrite stop-hook tests as a couple of FileCheck tests.
Rewrite stop-hook tests as a couple of FileCheck tests
Nov 16 2018, 3:10 PM