Skip to content

Commit

Permalink
[compiler] Remove dead OverloadInfo code
Browse files Browse the repository at this point in the history
This code came from an earlier development version of 7c64b47, and I
didn't realise it was effectively dead code.

All of the required overload info is being set by
mapGroupBuiltinToMuxGroupBuiltin, and analyzeBuiltin isn't ever required
to return overload info for language-level builtins.
  • Loading branch information
frasercrmck committed Aug 14, 2023
1 parent ab135b8 commit 447d97b
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions modules/compiler/utils/source/cl_builtin_info.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1006,7 +1006,6 @@ Builtin CLBuiltinInfo::analyzeBuiltin(Function const &Callee) const {

bool IsConvergent = false;
unsigned Properties = eBuiltinPropertyNone;
llvm::SmallVector<llvm::Type *, 2> OverloadInfo;
switch (ID) {
default:
// Assume convergence on unknown builtins.
Expand Down Expand Up @@ -1278,10 +1277,6 @@ Builtin CLBuiltinInfo::analyzeBuiltin(Function const &Callee) const {
case eCLBuiltinWorkgroupScanLogicalXorExclusive:
IsConvergent = true;
Properties |= eBuiltinPropertyMapToMuxGroupBuiltin;
if (ID != eCLBuiltinWorkgroupAll && ID != eCLBuiltinWorkgroupAny &&
ID != eCLBuiltinSubgroupAll && ID != eCLBuiltinSubgroupAny) {
OverloadInfo.push_back(Callee.getArg(0)->getType());
}
break;
}

Expand Down

0 comments on commit 447d97b

Please sign in to comment.