miyuki added inline comments.
================
Comment at: include/regex:3465
+ case '{':
+ case '}':
+ break;
----------------
Quuxplusone wrote:
> FWIW, I don't understand what's going on in this switch.
> Is it intentional that `'('` and `'|'` now take different paths here?
Yes. As far as I understand, only extended POSIX regular expressions support
alternation:
http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap09.html, so
"\\|" should be treated as error_escape
https://reviews.llvm.org/D42693
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
