[PATCH] D116327: [Coroutines] Enhance symmetric transfer for constant CmpInst

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 17:52:15 PST 2022


ChuanqiXu marked an inline comment as done.
ChuanqiXu added a comment.

In D116327#3234969 <https://reviews.llvm.org/D116327#3234969>, @rjmccall wrote:

> Hmm, alright.  I really don't love that high-level semantics are dependent on exactly the right transforms happening in exactly the right way, but I guess that's what the C++ committee has stuck us with.

Yeah, agreed. BTW, the symmetric transfer is not part of the C++ standard document like tail call is not in the standard too. It is just a compiler optimization that end-users are familiar with. So it becomes a somehow de facto standard so that users would feel very odd if the compiler fails to do so.


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

https://reviews.llvm.org/D116327



More information about the llvm-commits mailing list