This is an archive of the discontinued LLVM Phabricator instance.

[ORC][SimpleRemoteEPC] Fix android build error.
AbandonedPublic

Authored by sunho on Aug 1 2022, 4:40 PM.

Reviewers

lhames
sgraenitz

Summary

Fixes android build error. The compiler seems to be not capable of dealing with double macro expansion in this piece of code.

	Time	Test
	60,110 ms	x64 debian > AddressSanitizer-x86_64-linux-dynamic.TestCases::scariness_score_test.cpp
	60,110 ms	x64 debian > AddressSanitizer-x86_64-linux.TestCases::scariness_score_test.cpp

sunho requested review of this revision.Aug 1 2022, 4:40 PM

Fixed by sgraenitz using proper way!

Path

Size

llvm/

lib/

ExecutionEngine/

Orc/

2 lines

Show First 20 Lines • Show All 214 Lines • ▼ Show 20 Lines	LLVM_DEBUG({
case SimpleRemoteEPCOpcode::Result:		case SimpleRemoteEPCOpcode::Result:
dbgs() << "Result";		dbgs() << "Result";
assert(TagAddr.getValue() == 0 && "Non-zero TagAddr for Result?");		assert(TagAddr.getValue() == 0 && "Non-zero TagAddr for Result?");
break;		break;
case SimpleRemoteEPCOpcode::CallWrapper:		case SimpleRemoteEPCOpcode::CallWrapper:
dbgs() << "CallWrapper";		dbgs() << "CallWrapper";
break;		break;
default:		default:
llvm_unreachable("Invalid opcode");		assert(false && "Invalid opcode");
}		}
dbgs() << ", seqno = " << SeqNo		dbgs() << ", seqno = " << SeqNo
<< ", tag-addr = " << formatv("{0:x}", TagAddr.getValue())		<< ", tag-addr = " << formatv("{0:x}", TagAddr.getValue())
<< ", arg-buffer = " << formatv("{0:x}", ArgBytes.size())		<< ", arg-buffer = " << formatv("{0:x}", ArgBytes.size())
<< " bytes\n";		<< " bytes\n";
});		});
auto Err = T->sendMessage(OpC, SeqNo, TagAddr, ArgBytes);		auto Err = T->sendMessage(OpC, SeqNo, TagAddr, ArgBytes);
LLVM_DEBUG({		LLVM_DEBUG({
▲ Show 20 Lines • Show All 175 Lines • Show Last 20 Lines