[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin added a comment. In D129223#3633849 , @thakis wrote: > Reverted in e9fe20dab39edb911afacdb6ed9ec2c7499a59cf > for now. Thanks, I was going to do that when I noticed you alrea

[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. In D129223#3633754 , @thakis wrote: > This breaks check-clang: http://45.33.8.238/linux/80462/step_7.txt (Also on all other bots, e.g. https://lab.llvm.org/buildbot/#/builders/171/builds/17060 – did you run tests locally before

[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Reverted in e9fe20dab39edb911afacdb6ed9ec2c7499a59cf for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129223/new/ https://reviews.llvm.org/D129223 ___ cfe-commits mailing li

[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. This breaks check-clang: http://45.33.8.238/linux/80462/step_7.txt Please take a look and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129223/new/ https://reviews.llvm.org/D129223

[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Corentin Jabot via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbf45e27a676d: [Clang] Fix invalid utf-8 detection (authored by cor

[PATCH] D129223: [Clang] Fix invalid utf-8 detection

2022-07-06 Thread Corentin Jabot via Phabricator via cfe-commits
cor3ntin created this revision. Herald added a subscriber: hiraditya. Herald added a project: All. cor3ntin requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. The length of valid codepoints was incorrectly calculated which wa