Re: [PATCH] D12505: [X86] Set MaxVectorAlign for non-Darwin targets as well.

2016-09-27 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko added a comment. Looks like patch was not committed. https://reviews.llvm.org/D12505 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/list

Re: [PATCH] D12505: [X86] Set MaxVectorAlign for non-Darwin targets as well.

2015-09-09 Thread Eric Christopher via cfe-commits
echristo added a comment. (This ACK btw means "yes we should do this on all platforms", not that it's necessarily safe on any platform, see my mail on the original thread). -eric http://reviews.llvm.org/D12505 ___ cfe-commits mailing list cfe-comm

Re: [PATCH] D12505: [X86] Set MaxVectorAlign for non-Darwin targets as well.

2015-09-09 Thread Eric Christopher via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. This seems reasonable to me, I can't see why it wouldn't be ok. Thanks! -eric http://reviews.llvm.org/D12505 ___ cfe-commits mailing list c