[PATCH] D123776: [Support] Optimize (.*) regex matches

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 06:14:13 PDT 2022


xbolva00 added a comment.

In D123776#3452653 <https://reviews.llvm.org/D123776#3452653>, @nikic wrote:

> In D123776#3452587 <https://reviews.llvm.org/D123776#3452587>, @tstellar wrote:
>
>> Is there some reason we can't use the c++11 regex library for this?
>
> `std::regex` implementations are known to perform very badly for ABI compatibility reasons -- but it's possible that they're still better than this one, so I guess it would be worth trying at least.

Is it worth to spend time to port this matching code to std::regex when you could use much better libs?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123776/new/

https://reviews.llvm.org/D123776



More information about the llvm-commits mailing list