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> --- machxo2/arch.h | 15 +++++++++++---- machxo2/archdefs.h | 10 +++++++--- 2 files changed, 18 insertions(+), 7 deletions(-) (limited to 'machxo2') diff --git a/machxo2/arch.h b/machxo2/arch.h index 7a22dd91..15535fe1 100644 --- a/machxo2/arch.h +++ b/machxo2/arch.h @@ -18,9 +18,14 @@ * */ -#ifndef NEXTPNR_H -#error Include "arch.h" via "nextpnr.h" only. -#endif +#ifndef MACHXO2_ARCH_H +#define MACHXO2_ARCH_H + +#include + +#include "base_arch.h" +#include "nextpnr_namespaces.h" +#include "nextpnr_types.h" NEXTPNR_NAMESPACE_BEGIN @@ -37,7 +42,7 @@ template struct RelPtr const T *get() const { return 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()); } @@ -684,3 +689,5 @@ struct Arch : BaseArch }; NEXTPNR_NAMESPACE_END + +#endif /* MACHXO2_ARCH_H */ diff --git a/machxo2/archdefs.h b/machxo2/archdefs.h index 433b1b6b..31575487 100644 --- a/machxo2/archdefs.h +++ b/machxo2/archdefs.h @@ -18,9 +18,11 @@ * */ -#ifndef NEXTPNR_H -#error Include "archdefs.h" via "nextpnr.h" only. -#endif +#ifndef MACHXO2_ARCHDEFS_H +#define MACHXO2_ARCHDEFS_H + +#include "idstring.h" +#include "nextpnr_namespaces.h" NEXTPNR_NAMESPACE_BEGIN @@ -153,3 +155,5 @@ template <> struct hash }; } // namespace std + +#endif /* MACHXO2_ARCHDEFS_H */ -- cgit v1.2.3