This is an archive of the discontinued LLVM Phabricator instance.

Update root CMake to define LLVM_ENABLE_DIA_SDK when appropriate.
ClosedPublic

Authored by zturner on Feb 5 2015, 9:01 PM.

Details

Summary

This allows all CMake projects, as well as C++ code, to detect if and when DIA SDK is available for use.

This is independent of actually having any DIA-based code in the codebase, so in theory can go in now, even though the DIA stuff isn't in yet.

I'm not sure if I did this correctly. LMK if I overlooked something or if I'm going about this wrong.

Diff Detail

Repository
rL LLVM

Event Timeline

zturner updated this revision to Diff 19456.Feb 5 2015, 9:01 PM
zturner retitled this revision from to Update root CMake to define LLVM_ENABLE_DIA_SDK when appropriate..
zturner updated this object.
zturner edited the test plan for this revision. (Show Details)
zturner added a reviewer: chandlerc.
zturner added a subscriber: Unknown Object (MLST).

friendly ping to bring this to the top of your inbox

chandlerc added inline comments.Feb 9 2015, 5:01 PM
include/llvm/Config/config.h.in
540–542 ↗(On Diff #19456)

Need to update the config.h.cmake as well.

include/llvm/Config/llvm-config.h.cmake
83–85 ↗(On Diff #19456)

We shouldn't expose this to our headers, or our installed headers.

zturner updated this revision to Diff 19635.Feb 9 2015, 5:58 PM

Fix up suggestions from chandlerc. Moved the detection logic into cmake-ix as well, and renamed the variable HAVE_DIA_SDK instead of LLVM_ENABLE_DIA_SDK.

chandlerc accepted this revision.Feb 9 2015, 6:19 PM
chandlerc edited edge metadata.

LGTM, thanks.

This revision is now accepted and ready to land.Feb 9 2015, 6:19 PM
This revision was automatically updated to reflect the committed changes.