labath added a reviewer: JDevlieghere. labath added a subscriber: JDevlieghere. labath added a comment.
I like this approach, but as this is a novel idea, I'd like to get more opinions first. @JDevlieghere, what do you think? ================ Comment at: lldb/lit/lit.cfg.py:81 + cpuid_exe = lit.util.which('lit-cpuid', config.lldb_tools_dir) + if cpuid_exe is not None: + out, err, exitcode = lit.util.executeCommand([cpuid_exe]) ---------------- So if the binary is not found for any reason, this will just silently skip the relevant tests, right? Would it make sense to at least print a warning in this case? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61073/new/ https://reviews.llvm.org/D61073 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits