[PATCH] D38046: [AtomicExpandPass][X86] set MaxAtomicSizeInBitsSupported according to the target

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 19 14:00:14 PDT 2017


On Tue, Sep 19, 2017 at 11:35 AM, Eli Friedman via Phabricator <
reviews at reviews.llvm.org> wrote:

> efriedma added a comment.
>
> I would rather just fix clang (in lib/Basic/Targets/X86.cpp) to fix the
> bug, then handle the backend changes from https://reviews.llvm.org/D18201
> together.
>

I just feel backend has more target information to choose the precise
inline atomic size. But you are right, it is better to let D18201 to handle
it together.


>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D38046
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170919/01fdc352/attachment.html>


More information about the llvm-commits mailing list