[PATCH] D103119: [scudo] Get rid of initLinkerInitialized

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 16 10:55:14 PDT 2021


cryptoad added a comment.

In D103119#2822008 <https://reviews.llvm.org/D103119#2822008>, @yroux wrote:

> Hi,
>
> this is due to a misaligned data access, in Thumb the code generated for `unmapTestOnly`
> contains a Vector Store instruction `vst1.64 {d16-d17}, [r0, :128]` which needs a 16-bytes
> alignement, but the adress of `TSDs` (which is in `r0`) is not.

Thank you Yvan!
Potential fix: https://reviews.llvm.org/D104402
The new `DCHECK` caught the alignment issue on x86/64, I assume this would work for armhf as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103119



More information about the llvm-commits mailing list