[PATCH] D56155: [docs] cttz and ctlz return poison, not undef, when argument is 0

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 29 13:15:36 PST 2018


lebedev.ri added inline comments.


================
Comment at: docs/LangRef.rst:12251-12252
 the intrinsic should ensure that a zero as the first argument produces a
 defined result. Historically some architectures did not provide a
 defined result for zero values as efficiently, and many algorithms are
 now predicated on avoiding zero-value inputs.
----------------
Should `defined` also be reworded, since it is no longer 'un'`defined`, but `poison`?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56155





More information about the llvm-commits mailing list