[PATCH] D92162: [LangRef] Add poison constant

Fri Nov 27 23:45:09 PST 2020

aqjune added inline comments.

================
Comment at: llvm/docs/LangRef.rst:3660

+A poison value is a result of an erroneous operation.
 In order to facilitate speculative execution, many instructions do not
----------------
nikic wrote:
> Would personally drop this sentence, seems pretty vague.
(it was saved as a draft, submitting now) A high-level definition of poison was necessary IMO - so I put a topic sentence here. Since technical descriptions are below, I think it is okay to be conceptual here.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92162