search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

[ExceptionAnalyzer] Doesn't deal with CapturedStmt / CapturedDecl (within OpenMP directives) #40072

Open Quuxplusone opened 5 years ago

Quuxplusone commented 5 years ago
Bugzilla Link PR41102
Status NEW
Importance P enhancement
Reported by Roman Lebedev (lebedev.ri@gmail.com)
Reported on 2019-03-16 13:45:06 -0700
Last modified on 2019-03-22 06:52:01 -0700
Version unspecified
Hardware PC Linux
CC adam.balogh@ericsson.com, development@jonas-toth.eu, douglas_yung@playstation.sony.com, klimek@google.com, lebedev.ri@gmail.com
Fixed by commit(s)
Attachments
Blocks
Blocked by
See also 
https://godbolt.org/z/GpMpXF

int thrower() {
  throw 42;
}
void bad_for_header_XFAIL(const int a) {
#pragma omp for
  for (int i = 0; i < thrower(); i++)
    ;
  // FIXME: this really should be caught by bugprone-exception-escape.
}
Quuxplusone commented 5 years ago

Well yeah, bad_for_header_XFAIL() should be noexcept, but that doesn't change anything here.