<div dir="ltr">I know this is super confusing but `expectedFailureAll` doesn't yet support regular expressions.  I'm working on this code as we speak, so it will soon.  But what you're probably thinking of is `skipUnlessArch` (but ironically not `skipIfArch` or `expectedFailureArch`).  AHHHHHHHHH.<div><br></div><div>Anyway, that's why I'm fixing all this stuff, to make everything consistent.  For now if he's using expectedFailureAll (which is the recommended way), none of the keyword arguments will take a regex.</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 3, 2016 at 10:24 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">clayborg accepted this revision.<br>
clayborg added a comment.<br>
<br>
I believe we recently switched "archs" so that we auto detect the type so you could use a regular expression. See above inlined comments.<br>
<br>
Good to go unless you want to adopt any of my inlined suggestions.<br>
<br>
<br>
================<br>
Comment at: packages/Python/lldbsuite/test/functionalities/thread/crash_during_step/TestCrashDuringStep.py:24<br>
@@ -23,3 +23,3 @@<br>
     @expectedFailureAndroid("<a href="http://llvm.org/pr24497" rel="noreferrer" target="_blank">llvm.org/pr24497</a>", archs=['arm', 'aarch64'])<br>
-    @expectedFailureAll(archs=['mips', 'mipsel', 'mips64', 'mips64el'])    # IO error due to breakpoint at invalid address<br>
+    @expectedFailureAll(triple = 'mips')    # IO error due to breakpoint at invalid address<br>
     def test_step_inst_with(self):<br>
----------------<br>
You can use archs in this case if you want to by assigning it to a compiled regular expression:<br>
<br>
```<br>
@expectedFailureAll(archs=re.compile('^mips'))    # IO error due to breakpoint at invalid address<br>
```<br>
<br>
================<br>
Comment at: packages/Python/lldbsuite/test/functionalities/watchpoint/hello_watchlocation/TestWatchLocation.py:34<br>
@@ -33,3 +33,3 @@<br>
     @expectedFailureWindows("<a href="http://llvm.org/pr24446" rel="noreferrer" target="_blank">llvm.org/pr24446</a>") # WINDOWS XFAIL TRIAGE - Watchpoints not supported on Windows<br>
-    @expectedFailureAll(archs=['mips', 'mipsel', 'mips64', 'mips64el']) # Most of the MIPS boards provide only one H/W watchpoints, and S/W watchpoints are not supported yet<br>
+    @expectedFailureAll(triple = 'mips') # Most of the MIPS boards provide only one H/W watchpoints, and S/W watchpoints are not supported yet<br>
     def test_hello_watchlocation(self):<br>
----------------<br>
Could use:<br>
<br>
```<br>
@expectedFailureAll(archs=re.compile('^mips'))<br>
```<br>
<br>
================<br>
Comment at: packages/Python/lldbsuite/test/functionalities/watchpoint/step_over_watchpoint/TestStepOverWatchpoint.py:76<br>
@@ -75,3 +75,3 @@<br>
         arch = self.getArchitecture()<br>
-        if arch in ['mips', 'mipsel', 'mips64', 'mips64el']:<br>
+        if re.match("mips",arch):<br>
             self.runCmd("watchpoint delete 1")<br>
----------------<br>
should this be:<br>
<br>
```<br>
if re.match("^mips",arch):<br>
```<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D16840" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16840</a><br>
<br>
<br>
<br>
</blockquote></div>