dobbelaj-snps wrote:

> > Hmm. 10 changes + 1 new usage of setAAMetaData, But only 4 relevant changes 
> > in tests.. That part of SROA seems to lack some testing ?
> 
> Yes, will add the missing coverage, just wanted to make sure this makes sense 
> in general beforehand

I think it makes sense.

For this one to go through, I think we need at least a testcase that triggers 
the new 'PStore->setAAMetaData'... case. (line 4582)


https://github.com/llvm/llvm-project/pull/81289
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to