nealsid abandoned this revision.
nealsid added a comment.
Not needed with existing test coverage.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104437/new/
https://reviews.llvm.org/D104437
___
lldb-commi
nealsid added a comment.
Agreed, it looks like there's plenty of test coverage for this scenario, so
I'll revert this patch - sorry about that. I'm working on removing the
wchar/char preprocessor logic in Editline.cpp so I'll verify the existing
coverage before sending that for review. Thanks
Author: Muhammad Omair Javaid
Date: 2021-06-28T02:36:14Z
New Revision: 0f32d0034c7b9751b1ce7a89469328ac932faab8
URL:
https://github.com/llvm/llvm-project/commit/0f32d0034c7b9751b1ce7a89469328ac932faab8
DIFF:
https://github.com/llvm/llvm-project/commit/0f32d0034c7b9751b1ce7a89469328ac932faab8.di