https://github.com/bulbazord requested changes to this pull request.
I think you can probably just remove the check for `properties` value entirely.
This is the first property that may be printed, so it will never be greater
than 0. I think this will work:
```
stream.Printf(" ( arch=");
```
The other ones will have to stay because if the target architecture isn't
valid, properties may still be 0.
Alternatively, if you feel motivated, you can rewrite the target dumping code
altogether. My suggestion would be to gather all of the information up front
(triple, platform, pid, state, etc) and then print it all at the end in one
group so we don't have to do all these inline ternary conditions in the first
place.
https://github.com/llvm/llvm-project/pull/95675
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
