[all-commits] [llvm/llvm-project] bec04b: [ConstantHoisting] use struct rather than tuple fo...

Nick Desaulniers via All-commits all-commits at lists.llvm.org
Mon Jul 17 14:49:32 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bec04b4b00551672e0f381bfc627f0a29bb920bd
      https://github.com/llvm/llvm-project/commit/bec04b4b00551672e0f381bfc627f0a29bb920bd
  Author: Nick Desaulniers <ndesaulniers at google.com>
  Date:   2023-07-17 (Mon, 17 Jul 2023)

  Changed paths:
    M llvm/include/llvm/Transforms/Scalar/ConstantHoisting.h
    M llvm/lib/Transforms/Scalar/ConstantHoisting.cpp

  Log Message:
  -----------
  [ConstantHoisting] use struct rather than tuple for adjustments

We pack this info in a tuple just to spread it back out for a function
call. Spreads in C++ are awkward.  If I want to add an additional
element to the tuple, I need to add more calls to std::get<> later. Just
use a struct.

Reviewed By: void

Differential Revision: https://reviews.llvm.org/D155236




More information about the All-commits mailing list