[PATCH] D85228: [llvm-symbolizer] Add compatibility aliases for --inlining={true,false}

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 00:10:23 PDT 2020


jhenderson added a comment.

In D85228#2194243 <https://reviews.llvm.org/D85228#2194243>, @MaskRay wrote:

> @jhenderson This was not urgent... I wanted to give you some time for review but I accidentally committed it. Still happy to address your post-commit comments if any.

No worries. The change looks good to me anyway. One of the advantages in my mind with the new approach is that it is trivial to add new option aliases like this if we need to!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85228



More information about the llvm-commits mailing list