Lekensteyn added a comment. Except one thing, it looks reasonable to me. I'll try to run some tests and report back tomorrow.
(Not very familiar with Phabricator either. I still see some comments, hopefully the "Collapse" function does something useful here.) ================ Comment at: test/Driver/prefix-map.S:7 + +// More tests for this flag in debug-prefix-map.c. ---------------- Maybe restore the old file name (debug-prefix-map.S) since this still tests the debug prefix functionality? And otherwise this comment needs to be updated. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D49466/new/ https://reviews.llvm.org/D49466 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits