[all-commits] [llvm/llvm-project] 3178c2: [GVN] Move `ValueTable` out of `GVN.h` (NFC)

Momchil Velikov via All-commits all-commits at lists.llvm.org
Thu Sep 24 09:55:22 PDT 2026


  Branch: refs/heads/users/momchil-velikov/gvn-move-value-table
  Home:   https://github.com/llvm/llvm-project
  Commit: 3178c231a06c1d9f184095d32c3ce9163790be7f
      https://github.com/llvm/llvm-project/commit/3178c231a06c1d9f184095d32c3ce9163790be7f
  Author: Momchil Velikov <momchil.velikov at arm.com>
  Date:   2026-09-24 (Thu, 24 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)

  * Rename `GVNPass::ValueTable` to `GVNValueTable`, and move it out to
    the `llvm`  namepace 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