[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

This revision was automatically updated to reflect the committed changes. Closed by commit rG878ab6df033d: [TLI] Support for per-Function TLI that overrides available libfuncs (authored by tejohnson). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

tejohnson updated this revision to Diff 234090. tejohnson added a comment. Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67923/new/ https://reviews.llvm.org/D67923 Files: clang/lib/CodeGen/BackendUtil.cpp

[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

gchatelet accepted this revision. gchatelet added a comment. Thx ! Comment at: llvm/include/llvm/Analysis/TargetLibraryInfo.h:227 + const TargetLibraryInfoImpl , + LLVM_ATTRIBUTE_UNUSED Optional F = None) + : Impl(), OverrideAsUnavailable(NumLibFuncs) {

[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

hfinkel accepted this revision. hfinkel added a comment. This revision is now accepted and ready to land. In D67923#1784015 , @tejohnson wrote: > Please take a look. This is now updated to reflect the commit of D71193 > ,

[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

tejohnson added a comment. Please take a look. This is now updated to reflect the commit of D71193 , which translated the options to the new attributes. I also removed some comments that I realized didn't make sense, as we need to keep a baseline availability

[PATCH] D67923: [TLI] Support for per-Function TLI that overrides available libfuncs

tejohnson updated this revision to Diff 233841. tejohnson added a comment. Herald added subscribers: cfe-commits, hiraditya. Herald added a project: clang. Update after D71193 committed, converting -fno-builtin* options to attributes. Repository: rG LLVM