[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