[PATCH] D112916: Confusable identifiers detection

[Tom Stellard via Phabricator via cfe-commits]

tstellar accepted this revision.
tstellar added a comment.
This revision is now accepted and ready to land.

In D112916#3554158 <https://reviews.llvm.org/D112916#3554158>, @Mordante wrote:

> In D112916#3113140 <https://reviews.llvm.org/D112916#3113140>, @tstellar wrote:
>
>> The LLVM Board of Directors will do a legal review of this change.  We will give an update in 4-6 weeks.
>
> @tstellar, is there an update from the LLVM Board of Directors regarding using Unicode data?
> I'm implementing extended grapheme clustering in libc++ which requires using Unicode data.
> Note that MSVC STL (which is using the same license as the LLVM project) already uses Unicode data in the following generated header
> https://github.com/microsoft/STL/blob/main/stl/inc/__msvc_format_ucd_tables.hpp

Yes, it's fine to include the unicode data.


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

https://reviews.llvm.org/D112916