[PATCH] D78565: [clang][doc] Clang ARM CPU command line argument reference

[Sjoerd Meijer via Phabricator via cfe-commits]

SjoerdMeijer added a comment.

To be completely open about this, I had an offline chat with @psmith  and @kristof.beyls  about this. The reasoning is that this is probably related to my relatively poor choice of the 2 first cores that I describe here. They are microcontrollers, and Clang/LLVM doesn't provide a full embedded toolchain (linker, libraries, etc.), which is e.g. what we add downstream. So, if you're creating an embedded toolchain, you probably know what you're doing, and this is not one of the first problems that you have.

While I agree with that, I also mentioned that there are many issues orthogonal to this. Yes, we need a full toolchain, but I don't understand how improving things step-by-step can be a bad thing, or why we should wait for another thing to appear first before this becomes more valuable.  And yes, also -m option documentation needs to be improved, but again, for me that is orthogonal to this doc change. And what I also mentioned, is that absolutely nothing changes about the whole story here if you take an A-core application processor instead of an M-core, for which the toolchain probably looks simpler.

I agree with all your updates and replies on the cfe dev list. You mentioned "complete overhaul", and yes, I think option handling is inconsistent, undocumented, and so in general quite broken. Problem is that I didn't get a lot of buy in here and on the mailing list. So, I was happy to abandon this as I didn't feel like spending more time on this at this point.   But my idea was to step-by-step add more cores to this doc, also A-cores.


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

https://reviews.llvm.org/D78565