wmi added inline comments.
================ Comment at: lib/Basic/Targets/X86.h:898 + MaxAtomicPromoteWidth = 64; + MaxAtomicInlineWidth = 64; + } ---------------- efriedma wrote: > I don't think we need to mess with MaxAtomicPromoteWidth? > > Probably more intuitive to check "if (hasFeature" rather than "if > (!hasFeature". > > Adding a dedicated hook for this seems a bit overkill, but I don't have a > better suggestion. If 128 bits inline atomic is not supported, what is the point to promote atomic type to 128 bits? Repository: rL LLVM https://reviews.llvm.org/D38046 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
