[PATCH] D79317: Check type for forward reference definition

Wed May 6 06:57:46 PDT 2020

rengolin closed this revision.
rengolin added a comment.
Herald added a subscriber: stephenneuendorffer.

Ok, turns out the failure was a flaky test due to parallelism (CMake deps not right?), but the build finishes and the tests all pass.

I've pushed the change (5010b5b7e6c <https://reviews.llvm.org/rG5010b5b7e6cf0925465ecaa4111927ee9bcfac67>), as it has been approved and passes all test.

We'll discuss about the issues with the pre-merge tests in separate.

Thanks everyone!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79317