ahatanak added a comment.

I read the patches and review comments in https://reviews.llvm.org/D42776, but 
I don't remember why I added that assert. Maybe I was just trying to ensure the 
version number passed to `getTemporary` was the one that was used to create the 
temporary. For example, if a temporary was created using version X and later 
retrieved using version Y because of some bug in constant evaluation, the 
assertion would fail.

But the assertion is wrong, so maybe it should just be removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139713

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

Reply via email to