CaseyCarter / cmcstl2

An implementation of C++ Extensions for Ranges
Other
221 stars 68 forks source link

Fixes GCC 8.3 ICE #314

Closed cjdb closed 5 years ago

cjdb commented 5 years ago

Two commits:

makes DefaultConstructible to spec

DefaultConstructible wasn't to spec (no matter how much better off we would have been!) and was also triggering an ICE. Changing it match the spec seems to also have alleviated the ICE.

adds a cmcstl2 Dockerfile and updates the CI to directly use it

Cirrus CI has a mechanism to build a (cached) Dockerfile on demand instead of depending on a developer's unreliable e-mail filtering to prompt an update.

This commit is also useful for showing that ad60b813 passes on newer compilers.

tl;dr ad60b813 is a partial-fix for #311 and 5cfd9cdf confirms that by transferring the Dockerfile's ownership to the project.