aprantl added a comment. The confusion probably comes from the fact that the test is in CFE, but the change is in LLVM. Maybe it would make sense to **also** add a unittest. We have a couple for DIBuilder. http://reviews.llvm.org/D17884