<div dir="ltr"><div>> For example the type `const char *` also has a<span class="gmail-Apple-converted-space"> </span><br>summary provider, and I'd hope that this feature does not depend on the<span class="gmail-Apple-converted-space"> </span><br>specific way in which that summary is computed.<br></div><div><br></div>Yea, it would be nice to have this on Linux. Will try to use a type whose format is compatible across platforms</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Mi., 2. Nov. 2022 um 06:54 Uhr schrieb Pavel Labath <<a href="mailto:pavel@labath.sk">pavel@labath.sk</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On 01/11/2022 06:59, Michael Buch via lldb-commits wrote:<br>
> <br>
> Author: Michael Buch<br>
> Date: 2022-10-31T22:59:16-07:00<br>
> New Revision: 3e03873e363b5aa90e4488da63a6de0648d11aba<br>
> <br>
> URL: <a href="https://github.com/llvm/llvm-project/commit/3e03873e363b5aa90e4488da63a6de0648d11aba" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/3e03873e363b5aa90e4488da63a6de0648d11aba</a><br>
> DIFF: <a href="https://github.com/llvm/llvm-project/commit/3e03873e363b5aa90e4488da63a6de0648d11aba.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/3e03873e363b5aa90e4488da63a6de0648d11aba.diff</a><br>
> <br>
> LOG: [lldb][Test] Fix TestFrameFormatNameWithArgs.test on Windows/Linux<br>
> <br>
> * Windows doesn't support setting these breakpoints by basename<br>
> * On Linux std::function arguments aren't formatted as such<br>
<br>
The windows issue seems problematic (Zequan might be interested in <br>
that), but the non-pretty-printing of std::function is most likely <br>
caused by our lack of a pretty-printer for libstdc++'s (default for <br>
linux) std::function. It doesn't seem ideal that a fairly generic test <br>
would depend on the existence of a (very complicated) pretty printer.<br>
<br>
Is there a specific edge case that this particular check was trying to <br>
hit? Could it be moved into a separate test case, or ideally replaced by <br>
a something simpler? For example the type `const char *` also has a <br>
summary provider, and I'd hope that this feature does not depend on the <br>
specific way in which that summary is computed.<br>
<br>
> <br>
> Added:<br>
>      <br>
> <br>
> Modified:<br>
>      lldb/test/Shell/Settings/TestFrameFormatNameWithArgs.test<br>
> <br>
> Removed:<br>
>      <br>
> <br>
> <br>
> ################################################################################<br>
> diff  --git a/lldb/test/Shell/Settings/TestFrameFormatNameWithArgs.test b/lldb/test/Shell/Settings/TestFrameFormatNameWithArgs.test<br>
> index ab16c656624d8..d990114f57845 100644<br>
> --- a/lldb/test/Shell/Settings/TestFrameFormatNameWithArgs.test<br>
> +++ b/lldb/test/Shell/Settings/TestFrameFormatNameWithArgs.test<br>
> @@ -1,3 +1,4 @@<br>
> +# REQUIRES: system-darwin<br>
>   # RUN: %clangxx_host -g -O0 %S/Inputs/names.cpp -std=c++17 -o %t.out<br>
>   # RUN: %lldb -b -s %s %t.out | FileCheck %s<br>
>   settings set -f frame-format "frame ${function.name-with-args}\n"<br>
> <br>
> <br>
>          <br>
> _______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div>