efriedma added inline comments.
================ Comment at: lib/Basic/Targets/X86.h:898 + MaxAtomicPromoteWidth = 64; + MaxAtomicInlineWidth = 64; + } ---------------- wmi wrote: > 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? MaxAtomicPromoteWidth affects the ABI, so it can't vary based on the target CPU. 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
