dyung accepted this revision. dyung added a comment. In D105457#2878352 <https://reviews.llvm.org/D105457#2878352>, @nridge wrote:
> In D105457#2878302 <https://reviews.llvm.org/D105457#2878302>, @dyung wrote: > >> Interesting. If you can get me an updated patch, I can give it a try on my >> machine with 7.5 to verify if you like. > > I posted an updated patch, if you can give that a whirl that would be great! I can confirm that it builds and tests successfully on a machine with gcc 7.5. Assuming you have tested with more recent compilers and that it is fine there, LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105457/new/ https://reviews.llvm.org/D105457 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
