nikic 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. 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
