bjope created this revision.
bjope added a reviewer: rnk.
Herald added subscribers: cfe-commits, nhaehnle, jvesely.
Herald added a project: clang.
Use a forward declaration of DataLayout instead of including
DataLayout.h in clangs TargetInfo.h. This reduces include
dependencies toward DataLayout.h
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
It's interesting that Basic depends on IR at all.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69262/new/
https://reviews.llvm.org/D69262
__
This revision was automatically updated to reflect the committed changes.
Closed by commit rG78424e5f8417: Prune include of DataLayout.h from
include/clang/Basic/TargetInfo.h. NFC (authored by bjope).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D692