[all-commits] [llvm/llvm-project] a2a55d: Move llvm/Analysis/ObjCARCUtil.h to IR to fix laye...

David Blaikie via All-commits all-commits at lists.llvm.org
Thu Mar 4 16:16:59 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a2a55def354df2cd4de0b1cbd6b2795a07e6905a
      https://github.com/llvm/llvm-project/commit/a2a55def354df2cd4de0b1cbd6b2795a07e6905a
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2021-03-04 (Thu, 04 Mar 2021)

  Changed paths:
    R llvm/include/llvm/Analysis/ObjCARCUtil.h
    A llvm/include/llvm/IR/ObjCARCUtil.h
    M llvm/lib/IR/AutoUpgrade.cpp
    M llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
    M llvm/lib/Transforms/ObjCARC/ObjCARC.cpp
    M llvm/lib/Transforms/ObjCARC/ObjCARC.h
    M llvm/lib/Transforms/ObjCARC/ObjCARCContract.cpp
    M llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp
    M llvm/lib/Transforms/ObjCARC/PtrState.cpp
    M llvm/lib/Transforms/Utils/InlineFunction.cpp

  Log Message:
  -----------
  Move llvm/Analysis/ObjCARCUtil.h to IR to fix layering.

This is included from IR files, and IR doesn't/can't depend on Analysis
(because Analysis depends on IR).

Also fix the implementation - don't use non-member static in headers, as
it leads to ODR violations, inaccurate "unused function" warnings, etc.
And fix the header protection macro name (we don't generally include
"LIB" in the names, so far as I can tell).




More information about the All-commits mailing list