Re: [PATCH] D19827: Do not disable completely loop unroll when optimizing for size.

2016-05-04 Thread Marianne Mailhot-Sarrasin via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL268509: Do not disable completely loop unroll when optimizing for size. (authored by mamai). Changed prior to commit: http://reviews.llvm.org/D19827?vs=56133&id=56158#toc Repository: rL LLVM http://

Re: [PATCH] D19827: Do not disable completely loop unroll when optimizing for size.

2016-05-04 Thread Reid Kleckner via cfe-commits
rnk added a subscriber: rnk. rnk accepted this revision. rnk added a reviewer: rnk. rnk added a comment. This revision is now accepted and ready to land. lgtm To be clear, loop unrolling just lowers its size threshold when -Os is on: // Apply size attributes if (L->getHeader()->getParent()->

Re: [PATCH] D19827: Do not disable completely loop unroll when optimizing for size.

2016-05-04 Thread Marianne Mailhot-Sarrasin via cfe-commits
mamai retitled this revision from "Do not disable completely loop unroll according to optimization level." to "Do not disable completely loop unroll when optimizing for size.". mamai updated the summary for this revision. mamai updated this revision to Diff 56133. mamai added a comment. Modified