ioeric added a comment. In https://reviews.llvm.org/D49476#1167294, @akyrtzi wrote:
> Is it possible to add a regression test case ? I assume this is fixing some > issue, we should make sure we don't regress again. This fixes a downstream use case where we use `OrigD`. AFAICT, `c-index-test` is used to tests the index library, but it doesn't seem to use `OrigD` at all. We could probably add a flag to `c-index-test` to optionally print out `OrigD`. I didn't go down this path because I thought this change is trivial and doesn't break any existing changes. If you think optionally checking `OrigD` is reasonable, I'd be happy to add that with a test case. Repository: rC Clang https://reviews.llvm.org/D49476 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
