martong added a comment.

> We use NoEvalCall in all functions in this patch

I just now realized that it is actually not true, sorry about that. So, seems 
like `fnmatch`, `strcasecmp`, `strncasecmp` are `EvalCallAsPure`. I could skip 
them from this patch, if you guys consider this as dangerous and to bolster my 
previous comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82288



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

Reply via email to