chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land.
LGTM, again, really nice. Tiny tweak below. ================ Comment at: llvm/test/Other/available-externally-lto.ll:1 +; Ensure that we don't emit available_externally functions at -O2, unless -flto is present in which case we should preserve them for link-time inlining decisions. +; RUN: opt < %s -S -passes='default<O2>' | FileCheck %s ---------------- Wrap to 80-columns? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63580/new/ https://reviews.llvm.org/D63580 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
