[PATCH] D103859: [Attributor] Introduce an optimistic getUnderlyingObjects helper

Kuter Dinel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 16:04:10 PDT 2021


kuter accepted this revision.
kuter added a comment.
This revision is now accepted and ready to land.

Very small nit. LGTM



================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:357
+  auto StripCB = [&](Value *V) {
+    if (auto *UO = getUnderlyingObject(V))
+      return UO;
----------------
NIT: `getUnderlyingObject` never returns nullptr.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103859



More information about the llvm-commits mailing list