[PATCH] D87611: [SystemZ][z/OS] Set aligned allocation unavailable by default for z/OS

Fanbo Meng via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 15 08:33:02 PDT 2020


fanbo-meng added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:7216
   "aligned %select{allocation|deallocation}0 function of type '%1' is only "
   "available on %2 %3 or newer">;
+def err_aligned_allocation_unavailable_on_os : Error<
----------------
ahatanak wrote:
> 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"
> ```
> 
Thanks for reviewing Akira! This merged format does look better, I'll make the change.


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

https://reviews.llvm.org/D87611



More information about the cfe-commits mailing list