Aligned allocation is not supported on z/OS. This patch sets -faligned-alloc-unavailable as default in z/OS toolchain.
Details
Diff Detail
Event Timeline
Previous diff is incorrect and didn't include the entire change set, making a new one.
clang/lib/Driver/ToolChains/ZOS.cpp | ||
---|---|---|
25–27 | Please be consistent in using/not using additional namespace qualification. The declaration above this one takes advantage of the using directive for llvm::opt. | |
clang/lib/Sema/SemaExprCXX.cpp | ||
1841 | Just to ensure we are on the same page: |
clang/lib/Sema/SemaExprCXX.cpp | ||
---|---|---|
1841 | Yes, as we are trying to solve the same problem like Darwin (lack of support in system libraries), we use the same mechanism that are introduced for that. |
clang/include/clang/Basic/DiagnosticSemaKinds.td | ||
---|---|---|
7216–7219 | Is it possible to merge these two diagnostics? "aligned %select{allocation|deallocation}0 function of type '%1' is %select{only|not}4 " "available on %2 %select{%3 or newer|}4" |
clang/include/clang/Basic/DiagnosticSemaKinds.td | ||
---|---|---|
7216–7219 | Thanks for reviewing Akira! This merged format does look better, I'll make the change. |
clang/lib/Basic/Targets/OSTargets.h | ||
---|---|---|
773–774 | There is an extra semi-colon. |
clang/include/clang/Basic/DiagnosticSemaKinds.td | ||
---|---|---|
7217 | Thanks for catching that Hubert, I'll fix it. |
Can something be done about the trailing space?