[PATCH] D103489: [lld-macho][nfc] Remove unnecessary use of Optional<T*>

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 1 13:54:48 PDT 2021


int3 accepted this revision.
int3 added a comment.

I like the safety that `Optional<T*>` gives, but I'm also aware that we aren't consistent about using it in our codebase, so I'm fine with this


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103489



More information about the llvm-commits mailing list