[llvm] [LLVM][AutoUpgrade] Support default args on undeclared multi-call upgrades (PR #216246)
Rahul Joshi via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 28 09:07:54 PDT 2026
jurahul wrote:
My concern is that this is bundling 2 changes together as a way to test the new code. That is, migration of the collector_b immarg to use default value is a way to test the new changes and not required strictly. OTOH that migration cannot be done before the fix, since it will likely break autoupgrade for these intrinsics.
Would it make sense instead to add a temporary intrinsic to LLVM that uses a default value and do the AutoUpgrade fix and testing in that context? And keep adoption of MMA intrinsics to use this a separate change?
https://github.com/llvm/llvm-project/pull/216246
More information about the llvm-commits
mailing list