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

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

2015-08-31 Thread Ahmed Bougacha via cfe-commits
ab created this revision. ab added reviewers: echristo, rnk. ab added subscribers: alexr, dim, dschuff, tberghammer, cfe-commits. Follow-up to r246229: this isn't Darwin-specific, so let's enable it for all X86 targets. Quoting r160209, which introduced it: ``` Currently, if no aligned attribut