This is an archive of the discontinued LLVM Phabricator instance.

[LLDB][MIPS] Fix TestConcurrentEvents
ClosedPublic

Authored by nitesh.jain on Mar 23 2016, 3:35 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

nitesh.jain retitled this revision from to [LLDB][MIPS] Fix TestConcurrentEvents.
nitesh.jain updated this object.
nitesh.jain added reviewers: clayborg, ovyalov.
nitesh.jain set the repository for this revision to rL LLVM.
clayborg accepted this revision.Mar 23 2016, 9:57 AM
clayborg edited edge metadata.
This revision is now accepted and ready to land.Mar 23 2016, 9:57 AM
clayborg requested changes to this revision.Mar 23 2016, 10:00 AM
clayborg edited edge metadata.

Actually shouldn't this be:

triple = re.compile('^mips')

instead of:

triple = '^mips'
This revision now requires changes to proceed.Mar 23 2016, 10:00 AM

Hi Greg,

The regexes without usign re.compile will still work in python. So re.match("^mips","mips64") is also valid in python.

Python 2.7.3 (default, Jun 22 2015, 19:33:41)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import re
re.match("^mips","mips64")

<_sre.SRE_Match object at 0x7f7ccc67c030>

Hi Greg,

Please can you find some time to review this?

Thanks

clayborg accepted this revision.Apr 1 2016, 9:33 AM
clayborg edited edge metadata.
This revision is now accepted and ready to land.Apr 1 2016, 9:33 AM
This revision was automatically updated to reflect the committed changes.