[PATCH] D15272: [Verifier] Verifier that a GlobalValue is only used in this Module

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 23 10:12:49 PST 2015


> > How about having a single callback that takes a Value and do the cast
in the callback?
> My concern was that I already have to check for the type here in order to
see whether to call the callback or recurse, so I thought it was cleaner to
have two callbacks, but I don't feel very strongly about that.

Why not always call the callback? We have a set to avoid a loop.

Cheers, Rafael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151223/ce3cb78a/attachment.html>


More information about the llvm-commits mailing list