Page MenuHomePhabricator

Properly check for builtin
ClosedPublic

Authored by deadalnix on Dec 22 2017, 21:39.

Details

Summary

Previously, the function to check for symbols was used, but this is wrong as builtins are not symbols.

Test Plan
cmake -GNinja ..

Check that the configure file now has the builtins set to 1.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmakeclz2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1446
Build 1446: arc lint + arc unit

Event Timeline

schancel added inline comments.
src/config/CMakeLists.txt
60

This looks like we should put it in a cmake module?

This revision is now accepted and ready to land.Dec 26 2017, 08:13
This revision was automatically updated to reflect the committed changes.