[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