[all-commits] [llvm/llvm-project] 355725: [TSan] Fix missing inst cleanup (#144067)

[Kunqiu Chen via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 355725a25e6be38d7a97cab9e206d2a16a1bd849
      https://github.com/llvm/llvm-project/commit/355725a25e6be38d7a97cab9e206d2a16a1bd849
  Author: Kunqiu Chen <camsyn at foxmail.com>
  Date:   2025-06-18 (Wed, 18 Jun 2025)

  Changed paths:
    M llvm/lib/Transforms/Instrumentation/ThreadSanitizer.cpp
    M llvm/test/Instrumentation/ThreadSanitizer/atomic-non-integer.ll

  Log Message:
  -----------
  [TSan] Fix missing inst cleanup (#144067)

Commit 44e875ad5b2ce26826dd53f9e7d1a71436c86212 introduced a change that
replaces `ReplaceInstWithInst` with `Instruction::replaceAllUsesWith`,
without subsequent instruction cleanup.

This results in TSan leaving behind useless `load atomic` instructions
after 'replacing' them.

This commit adds cleanup back, consistent with the context.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications