https://github.com/JDevlieghere requested changes to this pull request.
As per Aiden's suggestion, please split this up into smaller PRs, grouped by
subproject.
https://github.com/llvm/llvm-project/pull/91856
___
cfe-commits mailing list
zufuliu wrote:
> Why do we need to make this change?
https://docs.python.org/3/reference/lexical_analysis.html#escape-sequences
> Changed in version 3.12: Unrecognized escape sequences produce a
> [SyntaxWarning](https://docs.python.org/3/library/exceptions.html#SyntaxWarning).
> In a future
e-kwsm wrote:
> Why do we need to make this change?
The valid escape sequences in Python are listed
[here](https://docs.python.org/3/reference/lexical_analysis.html#escape-sequences):
`\t`, `\n`, etc.
Invalid ones fixed here seem to be used for regular expression,
tstellar wrote:
Two questions:
* Did you use a script to generate this patch?
* Why do we need to make this change?
https://github.com/llvm/llvm-project/pull/91856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Eisuke Kawashima (e-kwsm)
Changes
---
Patch is 120.87 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/91856.diff
100 Files Affected:
- (modified)
llvmbot wrote:
@llvm/pr-subscribers-libcxx
@llvm/pr-subscribers-clang
Author: Eisuke Kawashima (e-kwsm)
Changes
---
Patch is 120.87 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/91856.diff
100 Files Affected:
- (modified)
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you,
