projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 6e9c589) | patch
Rollback assumeAligned in MemorySanitizer
authorGuillaume Chatelet <gchatelet@google.com>
Wed, 11 Dec 2019 10:00:32 +0000 (11:00 +0100)
committerGuillaume Chatelet <gchatelet@google.com>
Wed, 11 Dec 2019 13:25:21 +0000 (14:25 +0100)
commit3491109587146c987e5f2530cbbf609543c53206
treeb4ab3532506f052bc882557a9e8f7a59845d6fd2tree | snapshot
parent6e9c58946eeeebfe8eed7308a3b57611e225ad67commit | diff
Rollback assumeAligned in MemorySanitizer

Summary: Rollback of parts of D71213. After digging more into the code I think we should leave 0 when creating the instructions (CreateMemcpy, CreateMaskedStore, CreateMaskedLoad). It's probably fine for MemorySanitizer because Alignement is resolved but I'm having a hard time convincing myself it has no impact at all (although tests are passing).

Reviewers: courbet

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D71332
llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp diff | blob | history
Domain: System / Toolchain;