[PATCH] D95122: [Libcalls, Attrs] Annotate libcalls with noundef

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 21 07:35:01 PST 2021


nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.

I think this goes against the rules established in D87994 <https://reviews.llvm.org/D87994>, which allows memory accesses based on partially undefined values. Now, I think the change in D87994 <https://reviews.llvm.org/D87994> was a very bad one and we should undo it, but until then I don't think we can annotate with noundef here.


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

https://reviews.llvm.org/D95122



More information about the llvm-commits mailing list