pranavk wrote: Yes, I also think that this should be reverted. This commit seems to erroneously fail on cases like this:
https://github.com/tink-crypto/tink-cc/blob/4501627fe9ee312ad5d413600f050827b5f725ed/tink/util/secret_proto.h#L62 Note that ParseFromArray is being on an instance of T. So code is correct but this commit breaks such use cases. https://github.com/llvm/llvm-project/pull/84050 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits