[llvm] [TBAA] Do not rewrite TBAA tag in `adjustForAccess` if already exists (PR #96483)
Antonio Frighetto via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 6 02:10:27 PDT 2024
antoniofrighetto wrote:
> > > Is there any reason that clang shouldn't emit struct-path tags for !tbaa.struct?
> >
> > That seems to be the case, AFAICT, though this should be orthogonal wrt the above, if I'm not mistaken.
>
> I am not sure, have inconsistent/conflicting `tbaa` and `tbaa.struct` nodes seems like it could cause other problems as well. If `tbaa.struct` nodes would be consistent with the `!tbaa` nodes, this patch shouldn't be needed?
May there exist a specific reason why NewStructPathTBAA directly favours !tbaa metadata? This is at least what it seems to be from https://reviews.llvm.org/D41394. Maybe cc/ @kosarev?
https://github.com/llvm/llvm-project/pull/96483
More information about the llvm-commits
mailing list