[PATCH] D47854: [LangRef] Clarify semantics of load metadata.

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 7 07:18:31 PDT 2018


nlopes added a comment.

> Yes, although hopefully align(4) would have also been specified on the function argument. As the reference must have been bound to a valid object, from C++ semantics, we should know that it's properly aligned. Do we do that now?

Good point. It's not done right now by clang, no. But I guess it should so that we can fix this hoisting bug.


Repository:
  rL LLVM

https://reviews.llvm.org/D47854





More information about the llvm-commits mailing list