jyknight added inline comments.
================ Comment at: lib/Basic/Targets/RISCV.h:94 + if (HasA) + MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 32; + } ---------------- MaxAtomicPromoteWidth is an incompatible ABI-change kind of thing. We should set that to the maximum atomic size this target ABI will _EVER_ support with any hardware, since it changes the data layout for atomic types. MaxAtomicInlineWidth can change based on hardware, and be increased in the future if other hardware is introduced, but MaxAtomicPromoteWidth shouldn't. I think it should be set it to 64 for most 32-bit platforms, and 128 for most 64-bit platforms. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57450/new/ https://reviews.llvm.org/D57450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits