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"] + ---------------- omjavaid wrote: > 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. I still think ``` return self.getArchitecture().lower() == "aarch64" ``` Is the same thing unless you intend to expand the list. (but I understand what you're doing and either way works just a bit less neat IMO) 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
