[PATCH] D65108: Reland "driver: Don't warn about assembler flags being unused when not assembling"

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 12:40:31 PDT 2019


thakis marked an inline comment as done.
thakis added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:2051
+  // Claim flags for the integrated assembler only if it's being used.
+  if (IsIntegratedAs) {
+    if (UseRelaxAll(C, Args))
----------------
(This just wraps all the red lines on the lhs in an `if (IsIntegratedAs)` and leaves them all otherwise unchanged.)


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

https://reviews.llvm.org/D65108





More information about the cfe-commits mailing list