JDevlieghere marked 2 inline comments as done. JDevlieghere added a comment.
In https://reviews.llvm.org/D28768#647203, @malcolm.parsons wrote: > In https://reviews.llvm.org/D28768#647198, @JDevlieghere wrote: > > > In https://reviews.llvm.org/D28768#647177, @malcolm.parsons wrote: > > > > > What happens if the function has `auto` as the return type? > > > > > > Nothing, the constructor is left untouched. > > > Please add a test. I wanted to do that but it seems that the test script hard codes it to C++11, and the `auto` return type is a C++14 feature. Maybe I'm mistaken though, but I didn't manage to get it working, even when appending `-- -std=c++14` to the RUN command. Am I missing something? Repository: rL LLVM https://reviews.llvm.org/D28768 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
