From fe4608386eb163c70a75ed84beb07516af378b36 Mon Sep 17 00:00:00 2001 From: Keith Rothman <537074+litghost@users.noreply.github.com> Date: Fri, 12 Mar 2021 13:09:44 -0800 Subject: Split nextpnr.h to allow for linear inclusion. "nextpnr.h" is no longer the god header. Important improvements: - Functions in log.h can be used without including BaseCtx/Arch/Context. This means that log_X functions can be called without included "nextpnr.h" - NPNR_ASSERT can be used without including "nextpnr.h" by including "nextpnr_assertions.h". This allows NPNR_ASSERT to be used safely in any header file. - Types defined in "archdefs.h" are now available without including BaseCtx/Arch/Context. This means that utility classes that will be used inside of BaseCtx/Arch/Context can be defined safely in a self-contained header. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com> --- gowin/arch.h | 19 +++++++++++++++---- gowin/archdefs.h | 10 +++++++--- 2 files changed, 22 insertions(+), 7 deletions(-) (limited to 'gowin') diff --git a/gowin/arch.h b/gowin/arch.h index e0686d1c..0f975f77 100644 --- a/gowin/arch.h +++ b/gowin/arch.h @@ -18,9 +18,18 @@ * */ -#ifndef NEXTPNR_H -#error Include "arch.h" via "nextpnr.h" only. -#endif +#ifndef GOWIN_ARCH_H +#define GOWIN_ARCH_H + +#include +#include +#include +#include + +#include "base_arch.h" +#include "idstring.h" +#include "nextpnr_namespaces.h" +#include "nextpnr_types.h" NEXTPNR_NAMESPACE_BEGIN @@ -40,7 +49,7 @@ template struct RelPtr return const_cast(reinterpret_cast(reinterpret_cast(this) + offset)); } - const T &operator[](size_t index) const { return get()[index]; } + const T &operator[](std::size_t index) const { return get()[index]; } const T &operator*() const { return *(get()); } @@ -453,3 +462,5 @@ struct Arch : BaseArch }; NEXTPNR_NAMESPACE_END + +#endif /* GOWIN_ARCH_H */ diff --git a/gowin/archdefs.h b/gowin/archdefs.h index 963660c6..ef297d41 100644 --- a/gowin/archdefs.h +++ b/gowin/archdefs.h @@ -18,9 +18,11 @@ * */ -#ifndef NEXTPNR_H -#error Include "archdefs.h" via "nextpnr.h" only. -#endif +#ifndef GOWIN_ARCHDEFS_H +#define GOWIN_ARCHDEFS_H + +#include "idstring.h" +#include "nextpnr_namespaces.h" NEXTPNR_NAMESPACE_BEGIN @@ -68,3 +70,5 @@ struct ArchCellInfo }; NEXTPNR_NAMESPACE_END + +#endif /* GOWIN_ARCHDEFS_H */ -- cgit v1.2.3