jroelofs added a comment.

In https://reviews.llvm.org/D38599#893985, @danalbert wrote:

> In https://reviews.llvm.org/D38599#893903, @jroelofs wrote:
>
> > That reminds me... this does need a testcase or two.
>
>
> Oh, also, any test I add is going to fail, since the case I'm trying to 
> account for here is not the default behavior.


That's what an `available_feature` + `// REQUIRES:` is for.


https://reviews.llvm.org/D38599



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to