[PATCH] D74677: [bugpoint] Also replace declarations with null.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 25 13:25:32 PST 2020
fhahn planned changes to this revision.
fhahn added a comment.
In D74677#1891680 <https://reviews.llvm.org/D74677#1891680>, @reames wrote:
> 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.
My original intention with this patch was to just extend the existing behavior to declarations as well, but I agree using null is not ideal. I'll give your suggestion a try!
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