[PATCH] D104432: [Attributor] Introduce AAPointerInfo

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:1338
+      unsigned ArgNo = getIRPosition().getCallSiteArgNo();
+      ChangeStatus Changed;
+      if (ArgNo == 0) {
----------------
RKSimon wrote:
> @jdoerfert Static analysis is warning about an uninitialised variable - should this be ChangeStatus::UNCHANGED?
> 
> https://llvm.org/reports/scan-build/report-AttributorAttributes.cpp-handleAccess-10-6bf086.html#EndPath
Yes, will fix.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104432