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

2019-12-16 Thread Teresa Johnson via Phabricator via cfe-commits
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 https://reviews.llvm.org/D679

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

2019-12-16 Thread Teresa Johnson via Phabricator via cfe-commits
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 llvm/include/llvm/Ana

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

2019-12-15 Thread Guillaume Chatelet via Phabricator via cfe-commits
gchatelet accepted this revision. gchatelet added a comment. Thx ! Comment at: llvm/include/llvm/Analysis/TargetLibraryInfo.h:227 + const TargetLibraryInfoImpl &Impl, + LLVM_ATTRIBUTE_UNUSED Optional F = None) + : Impl(&Impl), OverrideAsUnavailable(NumLibFuncs) {

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

2019-12-13 Thread Hal Finkel via Phabricator via cfe-commits
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

2019-12-13 Thread Teresa Johnson via Phabricator via cfe-commits
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 ar

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

2019-12-13 Thread Teresa Johnson via Phabricator via cfe-commits
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 Gith