⚙ D56593 [SelectionDAG][RFC] Allow the user to specify a memeq function (v5). (original) (raw)

This is an archive of the discontinued LLVM Phabricator instance.

Table of Contentst-llvm/trunk/-trunk/-docs/-ReleaseNotes.rst-include/llvm/Transforms/Utils/-llvm/-Transforms/-Utils/-BuildLibCalls.h-lib/-Analysis/-TargetLibraryInfo.cpp-Transforms/Utils/-Utils/-BuildLibCalls.cpp-SimplifyLibCalls.cpp-test/-CodeGen/X86/-X86/-memcmp.ll-Transforms/-InferFunctionAttrs/-annotate.ll-InstCombine/-memcmp-1.ll-strcmp-1.llHide PanelfKeyboard Reference? Differential D56593 Authored by courbet on Jan 11 2019, 4:45 AM.Download Raw DiffReviewers jyknighthfinkel Commits rG8e16d73346f8: [SelectionDAG] Allow the user to specify a memeq function.rL355672: [SelectionDAG] Allow the user to specify a memeq function. Repository rL LLVM Event Timelinecourbet created this revision.Herald added a subscriber: fedor.sergeev. Harbormaster completed remote builds in B26694: Diff 181249.courbet added a reviewer: jyknight.Herald added a project: Restricted Project. courbet added a reviewer: hfinkel.xbolva00 added a subscriber: xbolva00.xbolva00 added inline comments.lib/Transforms/Utils/SimplifyLibCalls.cpp 917 ↗(On Diff #181249)gchatelet added inline comments.lib/Analysis/TargetLibraryInfo.cpp 54 ↗(On Diff #181249)Comment ActionsComment ActionsHarbormaster completed remote builds in B27733: Diff 185288.Comment ActionsHarbormaster completed remote builds in B27734: Diff 185289.Comment ActionsComment ActionsComment ActionsHerald added a subscriber: jdoerfert. Comment ActionsHarbormaster completed remote builds in B28044: Diff 186426.Comment ActionsComment ActionsComment ActionsMaskRay added a subscriber: MaskRay.Comment ActionsComment ActionsThis revision is now accepted and ready to land.Comment ActionsHerald added subscribers: Petar.Avramovic, kristina, atanasyan and 3 others. Comment ActionsHarbormaster completed remote builds in B28550: Diff 188369.Closed by commit rL355672: [SelectionDAG] Allow the user to specify a memeq function. (authored by courbet). This revision was automatically updated to reflect the committed changes.Comment ActionsComment ActionsComment ActionsComment ActionsComment Actionsbrzycki mentioned this in D64805: [AArch64] Expand bcmp() for small comparisons.evandro mentioned this in rL367898: [AArch64] Expand bcmp() for small block lengths.evandro mentioned this in rGa005c1ac4f3b: [AArch64] Expand bcmp() for small block lengths.hans mentioned this in rL368017: Merging r367898:.hansw mentioned this in rG13c43456a9a8: Merging r367898: --------------------------------------------------------------….Comment ActionsHerald added subscribers: ecnelises, pengfei. Comment ActionsComment ActionsComment Actionsgoldstein.w.n mentioned this in D127461: [SelectionDAG] Use __memcmpeq to replace bcmp and bool usage memcmp.Comment ActionsHerald added a project: Restricted Project. Herald added a subscriber: jsji. Comment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsComment ActionsFilesHistoryCommitsPathSizellvm/trunk/docs/ReleaseNotes.rst5 linesinclude/llvm/Transforms/Utils/BuildLibCalls.h4 lineslib/Analysis/TargetLibraryInfo.cpp13 linesTransforms/Utils/BuildLibCalls.cpp31 linesSimplifyLibCalls.cpp41 linestest/CodeGen/X86/memcmp.ll67 linesTransforms/InferFunctionAttrs/annotate.ll9 linesInstCombine/memcmp-1.ll21 linesstrcmp-1.ll24 linesDiffIDBaseDescriptionCreatedLintUnitBaseBaseDiff 1181249350207Jan 11 2019, 4:45 AM★★Diff 2185288350206Refactor emitBCmp() to BuildLibCalls.Feb 5 2019, 5:31 AM★★Diff 3185289353144rebaseFeb 5 2019, 5:46 AM★★Diff 4186426353674Add test to show that simplifylibcall composes (strcmp -> memcmp -> bcmp).Feb 12 2019, 2:21 AM★★Diff 5188369354013 - update realease notesFeb 26 2019, 6:57 AM★★Diff 6189823355671rL355672Mar 8 2019, 1:07 AM★★