[libcxx-commits] [PATCH] D63059: Implements multiline regex support

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Nov 18 09:14:42 PST 2020


Mordante marked an inline comment as done.
Mordante added a comment.

Thanks for the review!

In D63059#2400880 <https://reviews.llvm.org/D63059#2400880>, @ldionne wrote:

> LGTM, thanks a lot for this! Consider my comment about the comments you left behind, but feel free to commit without implementing my suggestion.

I'll remove them.

> I don't understand why CI isn't running -- it seems to be doing that with a few patches. If you commit this, can you please keep an eye on https://buildkite.com/llvm-project/libcxx-ci/builds?branch=master for the next build?

I'll keep an eye on the build bot.
I update the patches in the review using Phabricator's web-interface. Can the fact that I don't use arc be the reason the CI builds aren't triggered?


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

https://reviews.llvm.org/D63059



More information about the libcxx-commits mailing list