amaiorano added a comment. 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? 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
