steakhal added a comment. In D127742#3581874 <https://reviews.llvm.org/D127742#3581874>, @martong wrote:
> Why not replace all `getAs`? That's a massive change. I actually experimented with it and I see no advantage with that. > I suppose you picked conditions because `isa` can have variadic parameters. > If we are about to keep `getAs` then why not replace only those occurrences > where we see the benefit of the variadic `isa`? I believe it reads more naturally. `getAs<>` should be used only if we care about the value and don't know if it has one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127742/new/ https://reviews.llvm.org/D127742 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
