[PATCH] D143537: [Object][NFC] Factor out computeHeadersSize.

Jacek Caban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 10:20:28 PDT 2023


jacek added a comment.

In D143537#4213645 <https://reviews.llvm.org/D143537#4213645>, @mstorsjo wrote:

> FYI, most (all?) use of `llvm::Optional` has been migrated to `std::optional` these days.

Thanks, I wasn't sure which one I should use. I will update the patch.


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

https://reviews.llvm.org/D143537



More information about the llvm-commits mailing list