yingopq wrote: > > @nikic I checked several other arch-32 situations with ForceEnableInt128, > > and they all use `alloca i128, align 16`. But they did not modify the > > target datalayout content with `i128:128`, should mips be consistent? > > I think either way is fine here. Something to consider though, is that the > AutoUpgrade for the DataLayout has to match. So if you don't add `i128:128` > to the o32 DL, then it shouldn't get added by AutoUpgrade either.
OK, I decided to not add this. Please help review whether there has other issue, thanks! https://github.com/llvm/llvm-project/pull/112084 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
