rlibby added a comment. I am no expert here so I will defer, but I believe those suggestions are weaker assertions.
I believe it's really trying to assert that the NamedDecl type is the first template type in the point union, and is represented by a 0 bit in the addr, and that the pointer was appropriately aligned. So I tried to choose something that would fail if those were not true. I think that the dyn_cast<> approach will not fail appropriately if the NamedDecl type is not first. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71827/new/ https://reviews.llvm.org/D71827 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
