[PATCH] D147719: [OpenMP] Replace HeapToShared's initial value with `poison`

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 10:43:26 PDT 2023


jhuber6 added a comment.

This was in reponse to https://reviews.llvm.org/D147572 suggesting that we use `poison` in general over `undef`. I feel the semantics are roughly equivalent here, so I don't think this patch changes any existing behavior. All of these values get `addrspacecast`ed before they're used, but as far as I know those simply propagate poison values and don't imply any sort of volatility.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147719



More information about the llvm-commits mailing list