r-lib / bit64

An R package with an S3 Class for Vectors of 64bit Integers
32 stars 8 forks source link

Investigate validity of strict-aliasing compiler warning / workarounds #120

Open MichaelChirico opened 1 month ago

MichaelChirico commented 1 month ago

110 added some compiler warning suppression, but actually R CMD check complains about using those in a NOTE:

* checking pragmas in C/C++ headers and code ... NOTE
File which contains pragma(s) suppressing diagnostics:
  ‘src/integer64.c’

I will try to submit 4.6.0 with this NOTE intact, but it makes sense to revisit this warning later -- is it valid, does the suggested workaround break anything? It will take a bit more investigation to proceed.