[all-commits] [llvm/llvm-project] 5afba8: [GVN] use `AssertingVH` for leaders to improve com...

Princeton Ferro via All-commits all-commits at lists.llvm.org
Mon Apr 13 07:30:19 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5afba800906a5e69518abfb1995567c6186cfd6d
      https://github.com/llvm/llvm-project/commit/5afba800906a5e69518abfb1995567c6186cfd6d
  Author: Princeton Ferro <pferro at nvidia.com>
  Date:   2026-04-13 (Mon, 13 Apr 2026)

  Changed paths:
    M llvm/include/llvm/IR/ValueHandle.h
    M llvm/include/llvm/Transforms/Scalar/GVN.h
    M llvm/lib/Transforms/Scalar/GVN.cpp
    M llvm/unittests/IR/ValueHandleTest.cpp

  Log Message:
  -----------
  [GVN] use `AssertingVH` for leaders to improve compilation time (#175870)

Replace the manual check in `verifyRemoved()` with `AssertingVH`
instrumentation. For cases where the leader table becomes very large,
this is a cheaper way to verify we don't have dangling entries in the
leader table.

For this change, we must implement a move constructor for `AssertingVH`
so that we can keep the first entry as an inline-allocated node that
will be handled correctly as the table grows.



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


More information about the All-commits mailing list