[PATCH] D93927: [ArgPromotion] Copy !range metadata for loads.

Chenguang Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 15:37:57 PST 2021


wecing marked 2 inline comments as done.
wecing added inline comments.


================
Comment at: llvm/test/Transforms/ArgumentPromotion/metadata.ll:10
+;
+  %1 = call i32 @f_load_range({ i32, i32 }* %x)
+  ret i32 %1
----------------
fhahn wrote:
> Would a simple pointer type also work, or do we need a struct?
Thank you. Yes, a simple pointer is enough. The getelementptr is also unnecessary, so I also removed that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93927



More information about the llvm-commits mailing list