Skip to content

Commit

Permalink
[JITLink][AArch32] Fix MSVC error on missing constexpr capture
Browse files Browse the repository at this point in the history
The Windows MSVC builder complains: 'ImmMask' cannot be implicitly captured because no default capture mode has been
specified
When adding the explicit capture, Clang warns: Lambda capture 'ImmMask' is not required to be captured for this use

This seems to be a known implementation divergence. Using a default value capture should avoid the issue.
  • Loading branch information
weliveindetail committed Sep 13, 2023
1 parent be59265 commit 29f6c0f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion llvm/unittests/ExecutionEngine/JITLink/AArch32Tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ TEST(AArch32_Relocations, Arm_Call_Bare) {
0xffffffff, // ones
};

auto EncodeDecode = [](int64_t In, MutableWord &Mem) {
auto EncodeDecode = [=](int64_t In, MutableWord &Mem) {
Mem.patch(encodeImmBA1BlA1BlxA2(In), ImmMask);
return decodeImmBA1BlA1BlxA2(Mem.Wd);
};
Expand Down

0 comments on commit 29f6c0f

Please sign in to comment.