[PATCH] D37262: The issues with X86 prefixes: step 2

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 13 08:28:06 PDT 2017


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

Thanks for fixing that. Scope only applies to named objects, the unique_ptr here has no name. https://stackoverflow.com/questions/2298781/why-do-un-named-c-objects-destruct-before-the-scope-block-ends.

LGTM other than the one comment.


https://reviews.llvm.org/D37262





More information about the llvm-commits mailing list