[PATCH] D62532: [AIX] Implement function descriptor on SDAG

Dmitri Gribenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 7 02:27:46 PDT 2019


gribozavr added inline comments.


================
Comment at: llvm/trunk/include/llvm/MC/MCSymbolXCOFF.h:13
+#include "llvm/MC/MCSymbol.h"
+#include "llvm/IR/GlobalValue.h"
+
----------------
gribozavr wrote:
> llvm/IR/GlobalValue.h can't be included in MC, that creates a circular dependency between MC and IR libraries.  This circular dependency is causing an issue for build system that enforce layering. Is there another way to implement this change?
I committed a workaround in r362782, but it is only a workaround, and it would be nice to have a real fix.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62532/new/

https://reviews.llvm.org/D62532





More information about the llvm-commits mailing list