[PATCH] D87218: [builtins] Inline __paritysi2 into __paritydi2 and inline __paritydi2 into __parityti2.

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 7 16:41:45 PDT 2020


craig.topper added a comment.

In D87218#2259972 <https://reviews.llvm.org/D87218#2259972>, @MaskRay wrote:

> I think this is correct, but does their performance matter? `llvm/IR/RuntimeLibcalls.def` does not define them (they cannot be produced by llvm). Targets either emit popcount & 1 or detect the idiom and emit an optimized parity (x86 after PR46954)

Performance probably doesn't matter. Just one of the things a certain someone wasn't amused by.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87218/new/

https://reviews.llvm.org/D87218



More information about the cfe-commits mailing list