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

Thanks a lot!! This seems to take care of my issues.

Can you also include the following test case somewhere into, say, 
`test/Analysis/blocks-nrvo.c`? It crashes without this patch.

  // RUN: %clang_analyze_cc1 -w -analyzer-checker=core -fblocks -verify %s
  
  // expected-no-diagnostics
  
  typedef struct {
    int x;
  } S;
  
  void foo() {
    ^{
      S s;
      return s; // no-crash
    };
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109654

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to