[PATCH] D71799: [Attributor] AAUndefinedBehavior: Check for branches on undef value.

Stefanos Baziotis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 22 10:50:00 PST 2019


baziotis added a comment.

In D71799#1794405 <https://reviews.llvm.org/D71799#1794405>, @uenoku wrote:

>   if (!SimplifiedV.hasValue()) {
>     // No value yet, we can assume any value: assume this is undef BUT
>     // this is not _known_ so we don't put in the known set.
>   } else {
>     These are also assumption. You can't use these as known
>     if (undef) {
>       // insert in KnownUB 
>     } else {
>       // insert in NoUB.
>     }
>   }
>


Oh right, I have to call `ValueSimplifyAA.isKnown()`.


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

https://reviews.llvm.org/D71799





More information about the llvm-commits mailing list