[lldb][test_suite] change the test main.cpp to avoid expression reschedule

[kusmour]

Summary

Added a local variable to hold the return value to avoid delayed calculation on variables
This issue occurred when using NDK 20 clang with -O0 flag set

[labath]

The change is fine, but for my own education, could you elaborate on what this "delayed calculation" is, and how does it make the test fail?

[kusmour]

In D64769#1587318, @labath wrote:

The change is fine, but for my own education, could you elaborate on what this "delayed calculation" is, and how does it make the test fail?

The test set the breakpoint to the return statement, even thought the compiler flag -O0 was set, the first hit to that bp showed that g_common_1 = 0 which actually should be 21. The test assume when we hit the return statement, the value should already be calculated, but that's not the case for NDK20 clang.

So we reproduce the test manually, turned out that the value was calculated later. lldb first hit the return statement then jump back to line 20. By looking at the assembly that clang provided, it indeed jump back and forth.

It's possible that the jumping behavior is caused by optimization, considering there're some other tests having similar problem, and the behavior changed a lot from NDK 19 to 20. I made this change because this is irrelevant to testing global variables.

[labath]

In D64769#1587992, @kusmour wrote:

The test set the breakpoint to the return statement, even thought the compiler flag -O0 was set, the first hit to that bp showed that g_common_1 = 0 which actually should be 21. The test assume when we hit the return statement, the value should already be calculated, but that's not the case for NDK20 clang.

So we reproduce the test manually, turned out that the value was calculated later. lldb first hit the return statement then jump back to line 20. By looking at the assembly that clang provided, it indeed jump back and forth.

It's possible that the jumping behavior is caused by optimization, considering there're some other tests having similar problem, and the behavior changed a lot from NDK 19 to 20. I made this change because this is irrelevant to testing global variables.

Yeah, this problem isn't a bug in the debugger, but it could be considered a bug in the compiler, as one wouldn't expect the line numbers to jump around like this at -O0. And indeed, if we compare the the ndk r20 clang output with the current ToT, then we can see that they generate nearly identical assembly, but their line numbers differ. So it looks like somebody did consider this to be a bug and fixed it already.

Anyway, working around that here seems fine...

[davide]

That real

In D64769#1587992, @kusmour wrote:

In D64769#1587318, @labath wrote:

The change is fine, but for my own education, could you elaborate on what this "delayed calculation" is, and how does it make the test fail?

The test set the breakpoint to the return statement, even thought the compiler flag -O0 was set, the first hit to that bp showed that g_common_1 = 0 which actually should be 21. The test assume when we hit the return statement, the value should already be calculated, but that's not the case for NDK20 clang.

So we reproduce the test manually, turned out that the value was calculated later. lldb first hit the return statement then jump back to line 20. By looking at the assembly that clang provided, it indeed jump back and forth.

It's possible that the jumping behavior is caused by optimization, considering there're some other tests having similar problem, and the behavior changed a lot from NDK 19 to 20. I made this change because this is irrelevant to testing global variables.

that really sounds like a DebugInfo problem. Can you reproduce on ToT?

lldb/packages/Python/lldbsuite/test/lang/c/global_variables/main.c

 const char *g_file_global_cstr = "g_file_global_cstr";
 static const char *g_file_static_cstr = "g_file_static_cstr";
 int *g_ptr = &g_file_global_int;
 
 extern int g_a;
 int main (int argc, char const *argv[])
 {
     g_common_1 = g_file_global_int / g_file_static_int;
     static const char *g_func_static_cstr = "g_func_static_cstr";
     printf ("%s %s\n", g_file_global_cstr, g_file_static_cstr);
-    return g_file_global_int + g_a + g_common_1 + *g_ptr; // Set break point at this line.  //// break $source:$line; continue; var -global g_a -global g_global_int
+    int return_val = g_file_global_int + g_a + g_common_1 + *g_ptr;
+    return return_val; // Set break point at this line.  //// break $source:$line; continue; var -global g_a -global g_global_int
 }