[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