[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