This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGb4bb622fc94a: Also document -arch as -arch is mac specific
(authored by sylvestre.ledru).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST
serge-sans-paille accepted this revision.
serge-sans-paille added a comment.
This revision is now accepted and ready to land.
Thanks o/
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60380/new/
https://reviews.llvm.org/D60380
__
sylvestre.ledru updated this revision to Diff 405262.
sylvestre.ledru added a comment.
add .
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60380/new/
https://reviews.llvm.org/D60380
Files:
clang/docs/CommandGuide/clang.rst
Index: clang/docs/Co
sylvestre.ledru updated this revision to Diff 405259.
sylvestre.ledru added a comment.
Take in account Serge's comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60380/new/
https://reviews.llvm.org/D60380
Files:
clang/docs/CommandGuide/clang.
serge-sans-paille added inline comments.
Comment at: docs/CommandGuide/clang.rst:319
+
+ Specify the architecture to build for (Linux and others)
To be clear: this is probably misleading and should be `(all platforms)`
Repository:
rC Clang
CHANGES SINCE L
serge-sans-paille added a comment.
Reading through the source, it's relatively clear that `-arch` is specific to
the Mach0 format, but -target seems applicable to OSX too, see
https://godbolt.org/z/j8jvjbo84
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60380/ne
sylvestre.ledru created this revision.
sylvestre.ledru added a reviewer: rsmith.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D60380
Files:
docs/CommandGuide/clang.rst
Index: docs/CommandGuide/clang.rst
===