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
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
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
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.