compiler-rt CMake build currently tries to parse the triple and then
put it back together, but doing so inherently tricky, and doing so
from CMake is just crazy and currently doesn't handle triples that
have more than three components. Fortunatelly, the CMake really only
needs the architecture part, which is typically the first component,
to construct variants for other architectures. This means we can keep
the rest of the triple as is and avoid the parsing altogether.
Details
Diff Detail
- Repository
- rCRT Compiler Runtime
Event Timeline
compiler-rt/cmake/Modules/CompilerRTUtils.cmake | ||
---|---|---|
279 | I'm not sure if COMPILER_RT_DEFAULT_TARGET_TRIPLE_SUFFIX is the best name for what this represents (triple without the arch portion) so I'm open to better suggestions if you have any. |
compiler-rt/cmake/Modules/CompilerRTUtils.cmake | ||
---|---|---|
279 | I've done a bit more cleanup and this is no longer needed. |
compiler-rt/cmake/Modules/CompilerRTUtils.cmake | ||
---|---|---|
98 | Should this be begin + length - 1 since the foreach loop includes end in its range? | |
106 | Maybe break() after the first index that's too large? | |
319 | Do we need to implement list_sublist if this is the only use of it? Maybe removing the first element from triple_list would be simpler. |
Should this be begin + length - 1 since the foreach loop includes end in its range?