MaskRay added a comment.

I hope that we add just one option, instead of aliases. Aliases are usually 
added for compatibility when something is deprecated or changed, not for new 
options.

My "Request Changes" status still holds. The patch hasn't fixed something 
obvious: the description mentions print-supported-marchs (well, it should 
really name the exact name, `--print-supported-marchs`, don't omit dashes) 
while the implementation uses `...extensions`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146054

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to