ioeric added a comment. In https://reviews.llvm.org/D28943#651488, @amaiorano wrote:
> In https://reviews.llvm.org/D28943#651470, @ioeric wrote: > > > @amaiorano: The test itself is correct. It's just that this test failed in > > our internal test. We could've fixed it internally, but the fix would be > > ugly. Since the intended behavior is already covered in the case above it, > > and it's really just checking the default fallback style is LLVM, which is > > not related to the original change, I think it makes sense to get rid of > > the case. Hope you don't mind :) > > > Of course I don't mind :) Why did it fail your internal tests, btw? Just > curious. Was it something I could've detected myself? Probably not... it's just that our default fallback style is "Google" instead of "LLVM". Repository: rL LLVM https://reviews.llvm.org/D28943 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
