This is an archive of the discontinued LLVM Phabricator instance.

Fixed broken build under GCC 5.4.
ClosedPublic

Authored by sfuniak on Nov 26 2021, 6:13 PM.

Details

Summary

This diff fixes broken build caused by D108550. Under GCC 5, auto lambdas that capture this require this-> for member calls.

Diff Detail

Event Timeline

sfuniak created this revision.Nov 26 2021, 6:13 PM
sfuniak requested review of this revision.Nov 26 2021, 6:13 PM

Is there a bot to check to verify this fixes the gcc5 build?

bondhugula accepted this revision.Nov 26 2021, 7:32 PM
This revision is now accepted and ready to land.Nov 26 2021, 7:32 PM
This revision was automatically updated to reflect the committed changes.

@bondhugula I am not sure, but I built this with GCC 5.4 myself. The code compiled, and all the tests passed.

Is there a bot to check to verify this fixes the gcc5 build?

Yes, but on staging: https://lab.llvm.org/staging/#/builders/190