[PATCH] D32198: [TySan] A Type Sanitizer (LLVM)

Mon Apr 10 16:15:07 PDT 2023

leonardchan added a comment.

In D32198#4171275 <https://reviews.llvm.org/D32198#4171275>, @fhahn wrote:

> Address outstanding comments, remove some leftover commented-out code.
>
> In D32198#4162521 <https://reviews.llvm.org/D32198#4162521>, @leonardchan wrote:
>
>> In D32198#3929026 <https://reviews.llvm.org/D32198#3929026>, @fhahn wrote:
>>
>>> Commandeering after the recent updates to make review + follow-ups easier.
>>
>> @fhahn are there any updates with landing this? Is there any way I could help speed things up?
>
> The main things are help with reviewing the LLVM part (this patch) and testing the runtime part on different platforms (Linux x86/AArch64 if possible). I can do testing on macOS x86/ARM64.

Thanks. I've tested this patch alone on both linux x86 and aarch64 and can confirm they pass. I'll check the other ones next. Do you just need a rubber stamp lgtm on this patch?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D32198