[PATCH] D77058: [Clang] Add llvm.loop.unroll.disable to loops with -fno-unroll-loops.

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 31 13:44:48 PDT 2020


dexonsmith added a comment.

In D77058#1950019 <https://reviews.llvm.org/D77058#1950019>, @tejohnson wrote:

> I think this is a good approach, rather than a per-function attribute, since as mentioned this will be preserved through inlining.
>  @dexonsmith, does that seem reasonable to you? I missed the original patch and agree with you that we don't want to fix this in LTO by passing the option through to LTO.


SGTM!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77058





More information about the cfe-commits mailing list