Skip to content
This repository was archived by the owner on Mar 28, 2020. It is now read-only.

utils: add a helper class to lit for captured substitutions #81

Merged
merged 1 commit into from
Mar 20, 2018

Conversation

compnerd
Copy link
Collaborator

On Windows, if the substitution contains a back reference, it would
removed due to the replacement of the escape character in lit. Create a
helper class to avoid this which will simply ignore the replacement and
mark the substitution as having capture groups being referenced.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@327082 91177308-0d34-0410-b5e6-96231b3b80d8

On Windows, if the substitution contains a back reference, it would
removed due to the replacement of the escape character in lit. Create a
helper class to avoid this which will simply ignore the replacement and
mark the substitution as having capture groups being referenced.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@327082 91177308-0d34-0410-b5e6-96231b3b80d8
@compnerd
Copy link
Collaborator Author

CC: @bob-wilson @tkremenek ... could we pull this into stable please? It is needed to enable testing on Windows.

@tkremenek tkremenek merged commit 205f972 into apple:stable Mar 20, 2018
@compnerd compnerd deleted the substitute branch March 20, 2018 21:33
@compnerd
Copy link
Collaborator Author

Thanks!

@bob-wilson
Copy link
Contributor

I'll fix this one up, but for future reference, this should have gone to swift-4.2-branch. The stable branch is supposed to be a read-only "alias" for the current release branch.

@bob-wilson
Copy link
Contributor

(I'm also following up on whether the stable branch is locked like it is supposed to be: rdar://problem/38694144)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants