search

ldionne / metabench

A simple framework for compile-time benchmarks
metaben.ch
Other
181 stars 17 forks source link

added library Kvasir #168

Closed odinthenerd closed 7 years ago

odinthenerd commented 7 years ago

older clang failing, funny clang 3.7 with microsoft codegen works, guess I need to install clang 3.6. Is there a way to exclude tests on old clang for now ?

ldionne commented 7 years ago

Try looking at what we do for Hana on old GCCs

odinthenerd commented 7 years ago

looks like the porblem occurs with our wrappers around meta continuations. here is a short example of the problem https://godbolt.org/g/l2jkT8 I think we can solve this without too much trouble.

Seems harder to get clang 3.6 working on my machine ;)

brunocodutra commented 7 years ago

Reposting from cpplang for future reference:

working around the issue with the fixed type is easy, just make sure the nested alias template is dependent somehow, like this: https://godbolt.org/g/8dQoAc

odinthenerd commented 7 years ago

clang 3.5, 3.6, 3.7 will be slower because I force a lazy instantiation to get around the alias flaw but should work now

ldionne commented 7 years ago

If you're ready to merge, can you please squash into a sensible set of commits? Just edit your branch locally and then push-force. That will update the PR; no need to create a new PR.

brunocodutra commented 7 years ago

and update the copyright notices on top of each file

odinthenerd commented 7 years ago

OK ready to go, I'm always a bit timid when it comes to --force ;)