dpaoliello wrote:

> Need to make sure the autoupgraded string matches the new string, but 
> otherwise should be fine. (I remember last time we made major changes for 
> x86, the 128-bit integer alignment change, it was sort of tricky, but the 
> issue mostly wasn't the layout string itself.)

There are tests for this - I also hit an issue with an unrelated test due to 
data layout upgrading not working correctly, so I added this to the 
auto-upgrade tests as well.

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

Reply via email to