<div dir="ltr">Affirmative.<div><br><div>I've checked the compiler invocations before and after the patch. -g goes only to b.cpp. (also, this is the same pattern we use for other tests which want to build stuff with no debug info).</div></div></div><br><br><div class="gmail_quote"><div dir="ltr">On Wed, 21 Mar 2018 at 15:47, Davide Italiano <<a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Mar 21, 2018 at 2:43 AM, Pavel Labath via lldb-commits<br>
<<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> Author: labath<br>
> Date: Wed Mar 21 02:43:50 2018<br>
> New Revision: 328083<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=328083&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=328083&view=rev</a><br>
> Log:<br>
> Fix TestOperatorOverload for 32-bit builds<br>
><br>
> - use more goodies from Makefile.rules to correctly build a 32-bit<br>
> binary.<br>
> - avoid hardcoding typeof(nil) in the test.<br>
><br>
> This should partially fix the linux bot. There is still one assertion<br>
> failure remaining, which I'll have to investigate separately, as I am<br>
> not experiencing it locally.<br>
><br>
> Modified:<br>
>     lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile<br>
>     lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/TestOperatorOverload.py<br>
><br>
> Modified: lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile?rev=328083&r1=328082&r2=328083&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile?rev=328083&r1=328082&r2=328083&view=diff</a><br>
> ==============================================================================<br>
> --- lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile (original)<br>
> +++ lldb/trunk/packages/Python/lldbsuite/test/lang/cpp/operator-overload/Makefile Wed Mar 21 02:43:50 2018<br>
> @@ -1,18 +1,8 @@<br>
>  LEVEL = ../../../make<br>
><br>
>  CXX_SOURCES = a.cpp b.cpp<br>
> -CXXFLAGS_NO_DEBUGINFO = -c<br>
> -CXXFLAGS_DEBUGINFO = -c -g<br>
><br>
> -all: main<br>
> -<br>
> -main: a.o b.o<br>
> -       $(CXX) $^ -o $@ $(LDFLAGS)<br>
> +include $(LEVEL)/Makefile.rules<br>
><br>
>  a.o: a.cpp<br>
> -       $(CXX) $< $(CXXFLAGS_NO_DEBUGINFO) -o $@<br>
> -<br>
> -b.o: b.cpp<br>
> -       $(CXX) $< $(CXXFLAGS_DEBUGINFO) -o $@<br>
> -<br>
> -include $(LEVEL)/Makefile.rules<br>
> +       $(CC) $(CFLAGS_NO_DEBUG) -c $< -o $@<br>
><br>
<br>
Thanks for fixing this! Only one comment.<br>
I originally wrote this in a way that `-g` is passed only to b.cpp<br>
(and not to a).<br>
I needed that flag just to set a breakpoint (and I'm not sure if it's<br>
still hit, or just happens to work).<br>
Did this preserve the exact same meaning for the test?<br>
<br>
Best,<br>
<br>
--<br>
Davide<br>
</blockquote></div>