[PATCH] D39955: [Driver] Add command-line flags for the new TBAA metadata format

2017-11-15 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. Okay. I'm fine with living with both formats as long as we're not exposing this as a feature. Repository: rL LLVM https://reviews.llvm.org/D39955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D39955: [Driver] Add command-line flags for the new TBAA metadata format

2017-11-15 Thread Ivan Kosarev via Phabricator via cfe-commits
kosarev added a comment. Adding a (temporary) flag for the new format would help with updating tests that depend on the shape of TBAA metadata. There are lots of them and I guess some change frequently so we may want to address this in a few patches. That would also reduce the pressure on those

[PATCH] D39955: [Driver] Add command-line flags for the new TBAA metadata format

2017-11-13 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. We don't want this to become a long-term feature; we just want to switch the implementation over to the new format. Is there a good reason not to do that in one patch? If so, adding an option is acceptable, but it should be a -cc1 option only, because it is purely a

[PATCH] D39955: [Driver] Add command-line flags for the new TBAA metadata format

2017-11-13 Thread Ivan Kosarev via Phabricator via cfe-commits
kosarev created this revision. kosarev added a project: clang. This patch starts a series of changes to add support for the new TBAA metadata format proposed in this llvm-dev thread: http://lists.llvm.org/pipermail/llvm-dev/2017-November/118748.html Repository: rL LLVM https://reviews.llvm.