build: Drop AC_CHECK_DECLS for unused __builtin_clz #14747

pull Empact wants to merge 1 commits into bitcoin:master from Empact:builtin-clz changing 2 files +1 −5
  1. Empact commented at 4:16 PM on November 17, 2018: member

    __builtin_clz has never been used, decl check introduced in 4fd2d2fc97e21efceab849576e544160fd5e3e3d

  2. build: Drop AC_CHECK_DECLS for unused __builtin_clz
    Has never been used, introduced in 4fd2d2fc97e21efceab849576e544160fd5e3e3d
    a04230374c
  3. MarcoFalke added the label Build system on Nov 17, 2018
  4. MarcoFalke added the label Needs gitian build on Nov 17, 2018
  5. gmaxwell commented at 7:35 PM on November 17, 2018: contributor

    https://github.com/bitcoin/bitcoin/blob/master/src/crypto/common.h#L83

  6. Empact commented at 7:46 PM on November 17, 2018: member

    I see clzl and clzll are used, but not clz.

  7. DrahtBot commented at 10:51 AM on November 18, 2018: member

    <!--a722867cd34abeea1fadc8d60700f111-->

    Gitian builds for commit 35739976c1d9ad250ece573980c57e7e7976ae23 (master):

    Gitian builds for commit 5c83b871e312935550ca6c3c9f655209e99deeaa (master and this pull):

  8. DrahtBot removed the label Needs gitian build on Nov 18, 2018
  9. laanwj commented at 3:35 PM on November 21, 2018: member

    ~0 from me, it's somewhat likely clz will come in useful some time in the future

  10. MarcoFalke commented at 6:40 PM on November 22, 2018: member

    Too controversial for a trivial change. Closing for now.

  11. MarcoFalke closed this on Nov 22, 2018
  12. Empact deleted the branch on Nov 25, 2018
  13. MarcoFalke locked this on Sep 8, 2021
Contributors
EmpactgmaxwellDrahtBotlaanwjMarcoFalke
Labels
Build system
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-30 00:15 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me