HomePhabricator

[Utility] Reimplement RegularExpression on top of llvm::Regex

Authored by JDevlieghere on Aug 16 2019, 2:25 PM.

Description

[Utility] Reimplement RegularExpression on top of llvm::Regex

Originally I wanted to remove the RegularExpression class in Utility and
replace it with llvm::Regex. However, during that transition I noticed
that there are several places where need the regular expression string.
So instead I propose to keep the RegularExpression class and make it a
thin wrapper around llvm::Regex.

This patch also removes the workaround for empty regular expressions.
The result is that we are now (more or less) POSIX conformant.

Differential revision: https://reviews.llvm.org/D66174

llvm-svn: 369153

Details

Committed
JDevlieghereAug 16 2019, 2:25 PM
Differential Revision
D66174: [Utility] Reimplement RegularExpression on top of llvm::Regex
Parents
rG250aafa2c4a1: [X86] Support -mlong-double-80
Branches
Unknown
Tags
Unknown