libgccjit: add new version entry point
authorAndreaCorallo <andrea.corallo@arm.com>
Sun, 8 Mar 2020 13:46:33 +0000 (13:46 +0000)
committerAndrea Corallo <andrea.corallo@arm.com>
Tue, 31 Mar 2020 18:57:15 +0000 (20:57 +0200)
gcc/jit/ChangeLog
2020-03-31  Andrea Corallo  <andrea.corallo@arm.com>
    David Malcolm  <dmalcolm@redhat.com>

* docs/topics/compatibility.rst (LIBGCCJIT_ABI_13): New ABI tag
plus add version paragraph.
* libgccjit++.h (namespace gccjit::version): Add new namespace.
* libgccjit.c (gcc_jit_version_major, gcc_jit_version_minor)
(gcc_jit_version_patchlevel): New functions.
* libgccjit.h (LIBGCCJIT_HAVE_gcc_jit_version): New macro.
(gcc_jit_version_major, gcc_jit_version_minor)
(gcc_jit_version_patchlevel): New functions.
* libgccjit.map (LIBGCCJIT_ABI_13) New ABI tag.

gcc/testsuite/ChangeLog
2020-03-31  Andrea Corallo  <andrea.corallo@arm.com>

* jit.dg/test-version.c: New testcase.
* jit.dg/all-non-failing-tests.h: Add test-version.c.

gcc/jit/ChangeLog
gcc/jit/docs/topics/compatibility.rst
gcc/jit/libgccjit++.h
gcc/jit/libgccjit.c
gcc/jit/libgccjit.h
gcc/jit/libgccjit.map
gcc/testsuite/ChangeLog
gcc/testsuite/jit.dg/all-non-failing-tests.h
gcc/testsuite/jit.dg/test-version.c [new file with mode: 0644]

index 5d39b7bebc2158ef8615d576692cef9db0b6be27..d69a6a84c2c3928799b748bf407c42c606963f70 100644 (file)
@@ -1,3 +1,16 @@
+2020-03-31  Andrea Corallo  <andrea.corallo@arm.com>
+           David Malcolm  <dmalcolm@redhat.com>
+
+       * docs/topics/compatibility.rst (LIBGCCJIT_ABI_13): New ABI tag
+       plus add version paragraph.
+       * libgccjit++.h (namespace gccjit::version): Add new namespace.
+       * libgccjit.c (gcc_jit_version_major, gcc_jit_version_minor)
+       (gcc_jit_version_patchlevel): New functions.
+       * libgccjit.h (LIBGCCJIT_HAVE_gcc_jit_version): New macro.
+       (gcc_jit_version_major, gcc_jit_version_minor)
+       (gcc_jit_version_patchlevel): New functions.
+       * libgccjit.map (LIBGCCJIT_ABI_13) New ABI tag.
+
 2020-03-23  Andrea Corallo  <andrea.corallo@arm.com>
 
        * jit-playback.h
index a6faee0810e92b3552daa5eb516a67cb99455611..0c0ce070d722ec00736e2ae7b3caebb79bd72df2 100644 (file)
@@ -61,6 +61,28 @@ You can see the symbol tags provided by libgccjit.so using ``objdump``:
            LIBGCCJIT_ABI_0
    [...snip...]
 
+Programmatically checking version
+***************
+
+Client code can programmatically check libgccjit version using:
+
+.. function::  int gcc_jit_version_major (void)
+
+   Return libgccjit major version.  This is analogous to __GNUC__ in C code.
+
+.. function::  int gcc_jit_version_minor (void)
+
+   Return libgccjit minor version.  This is analogous to
+   __GNUC_MINOR__ in C code.
+
+.. function::  int gcc_jit_version_patchlevel (void)
+
+   Return libgccjit patchlevel version.  This is analogous to
+   __GNUC_PATCHLEVEL__ in C code.
+
+.. note:: These entry points has been added with ``LIBGCCJIT_ABI_13``
+          (see below).
+
 ABI symbol tags
 ***************
 
@@ -182,3 +204,14 @@ entrypoints:
 --------------------
 ``LIBGCCJIT_ABI_12`` covers the addition of
 :func:`gcc_jit_context_new_bitfield`
+
+``LIBGCCJIT_ABI_13``
+--------------------
+``LIBGCCJIT_ABI_13`` covers the addition of version functions via API
+entrypoints:
+
+  * :func:`gcc_jit_version_major`
+
+  * :func:`gcc_jit_version_minor`
+
+  * :func:`gcc_jit_version_patchlevel`
index 82a62d614c59e6eaf413901fb4a5bbdd19d7b035..69e67766640c97c618119f6815e5a38ce850255b 100644 (file)
@@ -49,6 +49,8 @@ namespace gccjit
   class timer;
   class auto_time;
 
+  namespace version {};
+
   /* Errors within the API become C++ exceptions of this class.  */
   class error
   {
@@ -1913,6 +1915,26 @@ auto_time::~auto_time ()
   m_timer.pop (m_item_name);
 }
 
+namespace version
+{
+inline int
+major_v ()
+{
+  return gcc_jit_version_major ();
+}
+
+inline int
+minor_v ()
+{
+  return gcc_jit_version_minor ();
+}
+
+inline int
+patchlevel_v ()
+{
+  return gcc_jit_version_patchlevel ();
+}
+} // namespace version
 } // namespace gccjit
 
 #endif /* #ifndef LIBGCCJIT_PLUS_PLUS_H */
index 83055fc297b8a61b6fb140f3ddb3cc062cf14939..a29e9885e59b641e0c1d3e479032f17ac881ef6f 100644 (file)
@@ -23,6 +23,8 @@ along with GCC; see the file COPYING3.  If not see
 #include "coretypes.h"
 #include "timevar.h"
 #include "typed-splay-tree.h"
+#include "cppbuiltin.h"
+#include <pthread.h>
 
 #include "libgccjit.h"
 #include "jit-recording.h"
@@ -3175,3 +3177,47 @@ gcc_jit_context_new_rvalue_from_vector (gcc_jit_context *ctxt,
      as_vec_type,
      (gcc::jit::recording::rvalue **)elements);
 }
+
+/* A mutex around the cached state in parse_basever.
+   Ideally this would be within parse_basever, but the mutex is only needed
+   by libgccjit.  */
+
+static pthread_mutex_t version_mutex = PTHREAD_MUTEX_INITIALIZER;
+
+struct version_info
+{
+  /* Default constructor.  Populate via parse_basever,
+     guarded by version_mutex.  */
+  version_info ()
+  {
+    pthread_mutex_lock (&version_mutex);
+    parse_basever (&major, &minor, &patchlevel);
+    pthread_mutex_unlock (&version_mutex);
+  }
+
+  int major;
+  int minor;
+  int patchlevel;
+};
+
+
+extern int
+gcc_jit_version_major (void)
+{
+  version_info vi;
+  return vi.major;
+}
+
+extern int
+gcc_jit_version_minor (void)
+{
+  version_info vi;
+  return vi.minor;
+}
+
+extern int
+gcc_jit_version_patchlevel (void)
+{
+  version_info vi;
+  return vi.patchlevel;
+}
index 21a0dc09b03bb13d9a9ef8d846455b93880f25d1..1c5a12e9c0158b1c2e0c6baf6b1961259ccb0115 100644 (file)
@@ -1487,6 +1487,22 @@ gcc_jit_context_new_rvalue_from_vector (gcc_jit_context *ctxt,
                                        size_t num_elements,
                                        gcc_jit_rvalue **elements);
 
+#define LIBGCCJIT_HAVE_gcc_jit_version
+
+/* Functions to retrive libgccjit version.
+   Analogous to __GNUC__, __GNUC_MINOR__, __GNUC_PATCHLEVEL__ in C code.
+
+   These API entrypoints were added in LIBGCCJIT_ABI_13; you can test for their
+   presence using
+     #ifdef LIBGCCJIT_HAVE_gcc_jit_version
+ */
+extern int
+gcc_jit_version_major (void);
+extern int
+gcc_jit_version_minor (void);
+extern int
+gcc_jit_version_patchlevel (void);
+
 #ifdef __cplusplus
 }
 #endif /* __cplusplus */
index 4514bd3aa33840c18fb9b16285251bb3c96842b1..6137dd4b4b03a25dfc751dadb4b8df1ee5413b91 100644 (file)
@@ -179,4 +179,11 @@ LIBGCCJIT_ABI_11 {
 LIBGCCJIT_ABI_12 {
   global:
     gcc_jit_context_new_bitfield;
-} LIBGCCJIT_ABI_11;
\ No newline at end of file
+} LIBGCCJIT_ABI_11;
+
+LIBGCCJIT_ABI_13 {
+  global:
+    gcc_jit_version_major;
+    gcc_jit_version_minor;
+    gcc_jit_version_patchlevel;
+} LIBGCCJIT_ABI_12;
\ No newline at end of file
index 008acc6e231735117a54ae3b6806d26576498b11..220789ffe1e537e483b03e96d670d922bce613a5 100644 (file)
@@ -1,3 +1,8 @@
+2020-03-31  Andrea Corallo  <andrea.corallo@arm.com>
+
+       * jit.dg/test-version.c: New testcase.
+       * jit.dg/all-non-failing-tests.h: Add test-version.c.
+
 2020-03-31  Iain Buclaw  <ibuclaw@gdcproject.org>
 
        PR d/94424
index af744192a73eb9f177bbf466876ef1dbb3b3937e..ad469dad6994177358f11089d39aada7b0e8a425 100644 (file)
 
 /* test-vector-types.cc: We don't use this, since it's C++.  */
 
+/* test-version.c */
+#define create_code create_code_version
+#define verify_code verify_code_version
+#include "test-version.c"
+#undef create_code
+#undef verify_code
+
 /* test-volatile.c */
 #define create_code create_code_volatile
 #define verify_code verify_code_volatile
@@ -392,6 +399,9 @@ const struct testcase testcases[] = {
   {"using_global",
    create_code_using_global,
    verify_code_using_global},
+  {"version",
+   create_code_version,
+   verify_code_version},
   {"volatile",
    create_code_volatile,
    verify_code_volatile}
diff --git a/gcc/testsuite/jit.dg/test-version.c b/gcc/testsuite/jit.dg/test-version.c
new file mode 100644 (file)
index 0000000..4338a00
--- /dev/null
@@ -0,0 +1,26 @@
+#include <stdlib.h>
+#include <stdio.h>
+
+#include "libgccjit.h"
+
+#include "harness.h"
+
+#ifndef LIBGCCJIT_HAVE_gcc_jit_version
+#error LIBGCCJIT_HAVE_gcc_jit_version was not defined
+#endif
+
+void
+create_code (gcc_jit_context *ctxt, void *user_data)
+{
+  /* Do nothing.  */
+}
+
+void
+verify_code (gcc_jit_context *ctxt, gcc_jit_result *result)
+{
+  if (!gcc_jit_version_major ())
+    fail ("Major version is zero");
+  /* Minor and patchlevel can be zero.  */
+  gcc_jit_version_minor ();
+  gcc_jit_version_patchlevel ();
+}