anton-afanasyev added a comment. In D58675#1449051 <https://reviews.llvm.org/D58675#1449051>, @thakis wrote:
> Looks like this landed without tests. Please add tests in a follow-up. > > Also, it looks like the flag is currently a cc1 flag. This should probably be > a CoreOption instead so that both the gcc-style and the cl-style drivers > expose it. Right now, users can't really use this if I read the patch right > (without using -Xclang, which is discouraged.) Ok, thanks, I'm to add tests in a follow-up. It works like `clang -ftime-trace main.cpp` now. Not sure I know where CoreOption is, but I'm to figure out it, thanks. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58675/new/ https://reviews.llvm.org/D58675 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
