omjavaid 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"] + ---------------- DavidSpickett wrote: > 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. This was intentional as I wanted to keep this helper checking for platform architecture regardless of ABI or endianess. For an ILP32 inferior or be inferior our Native* are same as normal aarch64. 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
