[PATCH] D76792: [InlineFunction] Update metadata on loads that are return values

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/InlineFunction.cpp:1168
       return nullptr;
     return VMI->second;
   };
----------------
Isn't this just `VMap.lookup(V)`?


================
Comment at: llvm/lib/Transforms/Utils/InlineFunction.cpp:1196
+    return MDNode::get(Context, MDB.createConstant(Int));
+  };
   for (auto &BB : *CalledFunction) {
----------------
`Num`


================
Comment at: llvm/test/Transforms/Inline/ret_load_metadata.ll:69
+}
+
+; CHECK: ![[NONNULL]] = !{i64 1}
----------------
Is there a test where the attribute is part of the callee and not the calls site?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76792