kazutakahirata wrote:

> Feel free to batch some of these changes together in a single PR!

Thanks!  It's becoming harder to find occurrences where I can migrate 
`PU.dyn_cast<T>` to `dyn_cast<T>(PU)` (not to be confused with 
`dyn_cast_if_present<T>(PU)`) because I have to convince myself that `PU` is 
never null.  Not every places does `!PU.isNull()` or a dereference of 
`cast<T>(PU)`.

https://github.com/llvm/llvm-project/pull/125335
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to