[PATCH] D145206: [NFC] Properly handle optional minor value for ArchInfo::Version

Tomas Matheson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 05:04:41 PST 2023


tmatheson added a comment.

For anyone reading, there's more context here about `std::optional::value()`: https://discourse.llvm.org/t/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor/63716

I'll add the assert back in D145371 <https://reviews.llvm.org/D145371>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145206



More information about the llvm-commits mailing list