[PATCH] D33084: [GISel]: Fix undefined behavior while accessing DefaultAction map

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 12 14:59:35 PDT 2017


dsanders accepted this revision.
dsanders added a comment.

Yep, LGTM.

One nit: We could avoid introducing the Optional<> by testing for LLT.isValid() instead of testing for whether the Optional has a value


Repository:
  rL LLVM

https://reviews.llvm.org/D33084





More information about the llvm-commits mailing list