jrtc27 added inline comments.

================
Comment at: llvm/lib/Support/TargetParser.cpp:341-344
+  }
+  if (ISAInfo.hasExtension("d"))
+    return "lp64d";
+  return "lp64";
----------------
makes me feel more comfortable inside, and also is a bit more like the old code 
where we did explicitly check 32 and 64 (though that instead fell back on the 
next step, rather than asserting).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105555

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

Reply via email to