DavidSpickett added inline comments.

================
Comment at: lldb/packages/Python/lldbsuite/test/lldbtest.py:1296
+        """Returns true if the architecture is AArch64."""
+        return self.getArchitecture().lower() in ["aarch64"]
+
----------------
This can be:
```
return self.getArchitecture().lower() == "aarch64"
```

Unless you're expecting "aarch64_be" or "aarch64_32" as well.
```
return "aarch64" in self.getArchitecture().lower()
```

Not sure if lldb just has the single name.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105483/new/

https://reviews.llvm.org/D105483

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to