zoecarver added a comment.
Herald added a subscriber: danielkiss.

There are a lot of different ways we could implement the feature. We may want 
to only enable it when `-march=native`, or maybe only in the unstable ABI, and 
maybe we want to support aligned pairs on some architectures. I think that's an 
important discussion to have but I'm not sure _this_ patch is the best place to 
have that discussion.

Even if we don't use this patch in the implementation I think it would still be 
a good utility to have. Here's what I suggest: I commit this, create another 
patch to add a builtin that exposes this API, and then open a libc++ patch with 
a _possible_ implementation. In that patch, we can discuss how we should 
actually implement the feature and after we have a consensus I can do the work 
to implement it. Any objections to that plan?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74918/new/

https://reviews.llvm.org/D74918



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to