[PATCH] D55709: [docs][ARM] Improve How to Cross Compile Compiler-rt Builtins For Arm

[Peter Smith via Phabricator via llvm-commits]

peter.smith created this revision.
peter.smith added reviewers: efriedma, compnerd, weimingz.
Herald added subscribers: kristof.beyls, javed.absar, dberris.

Some recent experience on llvm-dev pointed out some errors in the document such as:

- Assumption of ninja without -GNinja
- Use of --march rather than -march
- Problems with host include files when a multiarch setup was used
- Insufficient flags given to clang when acting as the assembler
- Instructions on using the cmake cache file BaremetalARM.cmake were incomplete

There was also insufficient troubleshooting information for when a step went wrong. I've made an effort to improve the document a bit based on this experience. Main changes:

- Fixed the problems described above
- Added a troubleshooting section
- Cleared up one "at time of writing" that is no longer a problem.

There are quite a few changes in the doc. I'm happy to break this up into smaller changes.


https://reviews.llvm.org/D55709

Files:
  docs/HowToCrossCompileBuiltinsOnArm.rst

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55709.178240.patch
Type: text/x-patch
Size: 12638 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181214/64fb891f/attachment.bin>