[PATCH] D67871: [Attributor] Use abstract call sites for call site callback

Hideto Ueno via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 25 09:34:04 PDT 2019


uenoku added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:597-598
+    const IRPosition &ACSArgPos = IRPosition::callsite_argument(ACS, ArgNo);
+    // Check if a coresponding argument was found or if it is on not associated
+    // (which can happen for callback calls).
+    if (ACSArgPos.getPositionKind() == IRPosition::IRP_INVALID)
----------------
I couldn't understand this. Could you show me an example?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67871/new/

https://reviews.llvm.org/D67871





More information about the llvm-commits mailing list