[PATCH] D74677: [bugpoint] Also replace declarations with null.

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 25 09:31:51 PST 2020


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

Calling null is (I think) full out UB.  This may be a more hostile reduction strategy than is warranted.  It's likely to result in extremely unstable reductions.

An alternative would be to pick a canonical callee (non-null declaration) and map all calls of a given signature to the same decl.

Also, I'm not opposed to this version if you really want, just pointing out a possible downside.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74677





More information about the llvm-commits mailing list