Remove the workaround llvm.41761? #25588

issue Chandlerooo openend this issue on July 11, 2022
  1. Chandlerooo commented at 3:45 pm on July 11, 2022: none

    Is your feature request related to a problem? Please describe. I notice there is a workaound for llvm.41761 with a TODO tag in coding.h: https://github.com/bitcoin/bitcoin/blob/master/src/leveldb/util/coding.h

    0// TODO(costan): Remove port::kLittleEndian and the fast paths based on
    1//               std::memcpy when clang learns to optimize the generic code, as
    2//               described in https://bugs.llvm.org/show_bug.cgi?id=41761
    

    This compiler bug is marked as fixed: https://bugs.llvm.org/show_bug.cgi?id=41761

    Describe the solution you’d like Shall the workaround be removed? Is this bug actually fixed?

  2. Chandlerooo added the label Feature on Jul 11, 2022
  3. MarcoFalke commented at 3:52 pm on July 11, 2022: member
    Thanks, but this will need to be fixed in https://github.com/google/leveldb
  4. MarcoFalke closed this on Jul 11, 2022

  5. fanquake removed the label Feature on Jul 11, 2022
  6. fanquake added the label Upstream on Jul 11, 2022
  7. bitcoin locked this on Jul 11, 2023

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: 2024-12-30 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me