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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
