[PATCH] D62112: [ORC] fix use-after-move. NFC

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 21 15:24:52 PDT 2019


nickdesaulniers added a comment.

> because is it a string reference? (that might be invalidated by the move), a return by value (that then means it's reference extending, which is best avoided unless really needed),

Do you have links or more info to these problems? I've never heard the term "reference extending" before.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62112





More information about the llvm-commits mailing list