dblaikie added a comment.

In D100567#2697415 <https://reviews.llvm.org/D100567#2697415>, @yonghong-song 
wrote:

> To answer one of your questions above, if there is a function definition 
> before, dyn_cast<llvm::Function>(...) will return nullptr. I tried starting 
> from "Value" class and found dyn_cast to llvm::ConstantExpr will result in a 
> non-null object, but I did not trace down further with subclasses of 
> llvm::ConstantExpr.

Might be worth understanding what's going on here before committing this patch, 
as it might not be the right direction.

(perhaps the right thing to do is to look at the AST expression rather than 
whatever IR entity is created for it?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100567

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to