[PATCH] D99696: [clang] NRVO: Improvements and handling of more cases.

Anh Tuyen Tran via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 3 20:13:13 PDT 2021


anhtuyen added a comment.

In D99696#2856370 <https://reviews.llvm.org/D99696#2856370>, @mizvekov wrote:

> @anhtuyen
>
> I pushed a DR with a preliminary fix for it: https://reviews.llvm.org/D105380
>
> This is not ready to merge, still have to add test cases and also decide between a pointed fix like this, or improving the ergonomics of `getDeclAlign` by returning possible failure.
>
> But it does fix your repro.

Thank you very much Matheus @mizvekov for a quick fix.
I have tried your patch, and it worked for my testcase. I will wait for the official patch when you commit the changes.
Thanks, again!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99696



More information about the cfe-commits mailing list