================
@@ -303,6 +377,24 @@ LogicalResult CIRABIRewriteContext::rewriteCallSite(
     if (!newCall->hasAttr(attr.getName()))
       newCall->setAttr(attr.getName(), attr.getValue());
 
+  // Layer llvm.signext / llvm.zeroext onto the new call's arg_attrs and
+  // res_attrs for Extend args/return.
+  bool needsArgAttrUpdate = false;
+  for (const ArgClassification &ac : fc.argInfos)
+    if (ac.kind == ArgKind::Extend || ac.kind == ArgKind::Ignore) {
----------------
andykaylor wrote:

Why is this looking for `Extend || Ignore` but the similar loop in 
`updateArgAttrs` only looks for `Extend`?

https://github.com/llvm/llvm-project/pull/195745
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to