[PATCH] D35802: XFAIL tests on SLES11

Marshall Clow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 31 07:37:07 PDT 2017


mclow.lists added a comment.

Can you give an example of linux distro names with spaces in it?
Other than that, this looks fine to me.



================
Comment at: utils/libcxx/test/target_info.py:193
         name, _, _ = platform.linux_distribution()
-        name = name.lower().strip()
         return name # Permitted to be None
----------------
A comment here would be helpful to future readers.


Repository:
  rL LLVM

https://reviews.llvm.org/D35802





More information about the llvm-commits mailing list