[llvm] r332161 - Requirements can have & in them!

Chris Matthews via llvm-commits llvm-commits at lists.llvm.org
Fri May 11 17:13:54 PDT 2018


Author: cmatthews
Date: Fri May 11 17:13:54 2018
New Revision: 332161

URL: http://llvm.org/viewvc/llvm-project?rev=332161&view=rev
Log:
Requirements can have & in them!

Lets escape those so the XML is valid!

Modified:
    llvm/trunk/utils/lit/lit/Test.py

Modified: llvm/trunk/utils/lit/lit/Test.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/Test.py?rev=332161&r1=332160&r2=332161&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/Test.py (original)
+++ llvm/trunk/utils/lit/lit/Test.py Fri May 11 17:13:54 2018
@@ -386,7 +386,7 @@ class Test:
         elif self.result.code == UNSUPPORTED:
             unsupported_features = self.getMissingRequiredFeatures()
             if unsupported_features:
-                skip_message = "Skipping because of: " + ", ".join(unsupported_features)
+                skip_message = escape("Skipping because of: " + ", ".join(unsupported_features))
             else:
                 skip_message = "Skipping because of configuration."
 




More information about the llvm-commits mailing list