[all-commits] [llvm/llvm-project] 5e1ed6: [GVN] Move `ValueTable` out of `GVN.h` (NFC) (#226...

Momchil Velikov via All-commits all-commits at lists.llvm.org
Tue Sep 29 02:07:51 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5e1ed67dbadba1b2bc254411cf78e61db51687d1
      https://github.com/llvm/llvm-project/commit/5e1ed67dbadba1b2bc254411cf78e61db51687d1
  Author: Momchil Velikov <momchil.velikov at arm.com>
  Date:   2026-09-29 (Tue, 29 Sep 2026)

  Changed paths:
    M llvm/include/llvm/Transforms/Scalar/GVN.h
    A llvm/include/llvm/Transforms/Scalar/GVNHoist.h
    A llvm/include/llvm/Transforms/Scalar/GVNSink.h
    A llvm/include/llvm/Transforms/Scalar/GVNValueTable.h
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/lib/Passes/PassBuilderPipelines.cpp
    M llvm/lib/Transforms/Scalar/GVN.cpp
    M llvm/lib/Transforms/Scalar/GVNHoist.cpp
    M llvm/lib/Transforms/Scalar/GVNSink.cpp

  Log Message:
  -----------
  [GVN] Move `ValueTable` out of `GVN.h` (NFC) (#226233)

* Rename `GVNPass::ValueTable` to `GVNValueTable`, and move it out to
the `llvm` namespace and to its own file `GVNValueTable.h`.
* Move `GVNPass::Expression` into `llvm::GVNValueTable`.
* Move `GVNHoistPass` and `GVNSinkPass` to their own headers.

With these changes `GVHoist.cpp` and `GVNSink.cpp` no longer need to
include `GVN.h` or peek into `GVNPass` internals.



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