[all-commits] [llvm/llvm-project] 349110: Rollback assumeAligned in MemorySanitizer

Guillaume Chatelet via All-commits all-commits at lists.llvm.org
Wed Dec 11 05:32:24 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 3491109587146c987e5f2530cbbf609543c53206
      https://github.com/llvm/llvm-project/commit/3491109587146c987e5f2530cbbf609543c53206
  Author: Guillaume Chatelet <gchatelet at google.com>
  Date:   2019-12-11 (Wed, 11 Dec 2019)

  Changed paths:
    M llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp

  Log Message:
  -----------
  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




More information about the All-commits mailing list