[PATCH] D84398: [yaml2obj] - Introduce a way to set default values when using ShOffset, ShName and ShSize fields.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 23 08:39:44 PDT 2020


MaskRay added a comment.

The None proposal looks good to me. My only minor concern is how we differentiate it from a value called `None`. If we can't find other suitable syntactic form, then `=None` seems fine.


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

https://reviews.llvm.org/D84398





More information about the llvm-commits mailing list