[all-commits] [llvm/llvm-project] e82fcd: [Coroutines] Move util headers to include/llvm (#1...

[Tyler Nowicki via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e82fcda1475b6708b7d314fd7a54e551306d5739
      https://github.com/llvm/llvm-project/commit/e82fcda1475b6708b7d314fd7a54e551306d5739
  Author: Tyler Nowicki <tyler.nowicki at amd.com>
  Date:   2024-10-09 (Wed, 09 Oct 2024)

  Changed paths:
    A llvm/include/llvm/Transforms/Coroutines/ABI.h
    A llvm/include/llvm/Transforms/Coroutines/CoroInstr.h
    A llvm/include/llvm/Transforms/Coroutines/CoroShape.h
    M llvm/include/llvm/Transforms/Coroutines/CoroSplit.h
    A llvm/include/llvm/Transforms/Coroutines/MaterializationUtils.h
    A llvm/include/llvm/Transforms/Coroutines/SpillUtils.h
    A llvm/include/llvm/Transforms/Coroutines/SuspendCrossingInfo.h
    R llvm/lib/Transforms/Coroutines/ABI.h
    M llvm/lib/Transforms/Coroutines/CoroEarly.cpp
    M llvm/lib/Transforms/Coroutines/CoroFrame.cpp
    R llvm/lib/Transforms/Coroutines/CoroInstr.h
    M llvm/lib/Transforms/Coroutines/CoroInternal.h
    R llvm/lib/Transforms/Coroutines/CoroShape.h
    M llvm/lib/Transforms/Coroutines/CoroSplit.cpp
    M llvm/lib/Transforms/Coroutines/Coroutines.cpp
    M llvm/lib/Transforms/Coroutines/MaterializationUtils.cpp
    R llvm/lib/Transforms/Coroutines/MaterializationUtils.h
    M llvm/lib/Transforms/Coroutines/SpillUtils.cpp
    R llvm/lib/Transforms/Coroutines/SpillUtils.h
    M llvm/lib/Transforms/Coroutines/SuspendCrossingInfo.cpp
    R llvm/lib/Transforms/Coroutines/SuspendCrossingInfo.h
    M llvm/unittests/Transforms/Coroutines/ExtraRematTest.cpp

  Log Message:
  -----------
  [Coroutines] Move util headers to include/llvm (#111599)

Plugin libraries that use coroutines can do so right now, however, to
provide their own ABI they need to be able to use various headers, some
of which such are required (such as the ABI header). This change exposes
the coro utils and required headers by moving them to
include/llvm/Transforms/Coroutines. My experience with our out-of-tree
plugin ABI has been that at least these headers are needed. The headers
moved are:
* ABI.h (ABI object)
* CoroInstr.h (helpers)
 * Coroshape.h (Shape object)
 * MaterializationUtils.h (helpers)
 * SpillingUtils.h (helpers)
 * SuspendCrossingInfo.h (analysis)

This has no code changes other than those required to move the headers
and these are:
 * include guard name changes
 * include path changes
 * minor clang-format induced changes
 * removal of LLVM_LIBRARY_VISIBILITY



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