lei137 wrote: > I'm not an expert here, but I think the change you made makes sense since the > `Ty` is supposed to be the first field. > > Also, might be a dumb question, I saw we have 32-bit Linux run lines, but is > that still worth testing?
Thanks for taking a look @amy-kwan. The 32bit linux run line was added in response to review comment from Hubert: https://github.com/llvm/llvm-project/pull/101738#issuecomment-2268003106 https://github.com/llvm/llvm-project/pull/101738 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
