[PATCH] D97823: Update default arch on AIX

[Steven Wan via Phabricator via cfe-commits]

stevewan added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:403
+    if (T.isOSAIX()) {
+      unsigned major, minor, micro;
+      T.getOSVersion(major, minor, micro);
----------------
jsji wrote:
> `micro` unsed.
Sorry I missed this comment. I agree that the unused `micro` is a bit ugly, do you have a preference on how you'd like it to be improved? We don't have an existing query for only `major` and `minor`, and creating one seems unnecessary IMHO. If you think renaming the variable to `unused_micro` mitigates the problem, I can do that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97823