This is an archive of the discontinued LLVM Phabricator instance.

[COFF] Don't error out on undefined references to __enclave_config
ClosedPublic

Authored by mstorsjo on Dec 11 2017, 12:54 PM.

Details

Summary

This is required for linking the CRT from MSVC 2017 15.5.

While this is unrelated to guardcf.test in general, it's the same kind of issue (avoiding errors due to undefined symbols), so I added the symbol to that test instead of creating a completely new one.

Diff Detail

Repository
rL LLVM

Event Timeline

mstorsjo created this revision.Dec 11 2017, 12:54 PM
ruiu edited edge metadata.Dec 11 2017, 12:58 PM

What does this symbol do in MSVC 2017?

In D41089#951580, @ruiu wrote:

What does this symbol do in MSVC 2017?

I'm not quite sure - I just ran into it after updating to 15.5 (which is the latest update to 2017, released last week). I think the symbol is related to Intel SGX things; googling it landed me here: https://www.reddit.com/r/cpp/comments/6zjub9/msvc_now_has_partial_twophase_name_lookup/dn13dxz/

ruiu accepted this revision.Dec 11 2017, 1:16 PM

LGTM

Got it. We can figure this out later.

This revision is now accepted and ready to land.Dec 11 2017, 1:16 PM
This revision was automatically updated to reflect the committed changes.