[PATCH] D153582: [SystemZ][z/OS] Add required options/macro/etc for z/os compilation step

fanbo-meng accepted this revision. fanbo-meng added a comment. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153582/new/ https://reviews.llvm.org/D153582 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D145158: Make clang/test/C/C2x/n2934.c compatible with targets that do not support thread_local storage.

This revision was automatically updated to reflect the committed changes. Closed by commit rGbe646ef39298: Make clang/test/C/C2x/n2934.c compatible with targets that do not support… (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D145158: Make clang/test/C/C2x/n2934.c compatible with targets that do not support thread_local storage.

fanbo-meng created this revision. Herald added a project: All. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D145158 Files: clang/test/C/C2x/n2934.c Index:

[PATCH] D134034: [test] Use host platform specific error message substitution

fanbo-meng accepted this revision. fanbo-meng added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134034/new/ https://reviews.llvm.org/D134034

[PATCH] D108998: [SystemZ][z/OS] Create html report file with text flag

This revision was automatically updated to reflect the committed changes. Closed by commit rGae206db2d653: [SystemZ][z/OS] Create html report file with text flag (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108998/new/

[PATCH] D108998: [SystemZ][z/OS] Create html report file with text flag

fanbo-meng created this revision. Herald added a subscriber: martong. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Change OF_None to OF_Text flag in file creation, same reasoning as https://reviews.llvm.org/D97785

[PATCH] D108819: [MCParser][z/OS] Mark test as unsupported for the z/OS Target

This revision was automatically updated to reflect the committed changes. Closed by commit rG9d7a77c26d2f: [MCParser][z/OS] Mark test as unsupported for the z/OS Target (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D108819: [MCParser][z/OS] Mark test as unsupported for the z/OS Target

fanbo-meng created this revision. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Marking test as unsupported for the same reason as https://reviews.llvm.org/D105204 Repository: rG LLVM Github Monorepo

[PATCH] D107565: Revert "[SystemZ][z/OS] Update target specific __attribute__((aligned)) value for test"

This revision was automatically updated to reflect the committed changes. Closed by commit rG91e399519580: Revert [SystemZ][z/OS] Update target specific __attribute__((aligned)) value… (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D107565: Revert "[SystemZ][z/OS] Update target specific __attribute__((aligned)) value for test"

fanbo-meng created this revision. Herald added a subscriber: krytarowski. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This reverts commit d91234b21c1a1a34d98157089a8769d8f9a32f06

[PATCH] D107189: [z/OS]Remove overriding default attribute aligned value

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbdf4c7b738ee: [z/OS]Remove overriding default attribute aligned value (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D107189: [z/OS]Remove overriding default attribute aligned value

fanbo-meng created this revision. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Make DefaultAlignForAttributeAligned consistent with SystemZ. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D107189 Files:

[PATCH] D106890: [z/OS] Make MinGlobalAlign consistent with SystemZ

This revision was automatically updated to reflect the committed changes. Closed by commit rGa2d4b064644b: [z/OS] Make MinGlobalAlign consistent with SystemZ (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106890/new/

[PATCH] D106890: [z/OS] Make MinGlobalAlign consistent with SystemZ

fanbo-meng created this revision. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Remove overriding MinGlobalAlign to 0 for z/OS target to be consistent with SystemZ. Repository: rG LLVM Github Monorepo

[PATCH] D99514: [NFC] clang-formatting zos-alignment.c

This revision was automatically updated to reflect the committed changes. Closed by commit rGbd8dd580ffd2: [NFC] clang-formatting zos-alignment.c (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99514/new/

[PATCH] D99514: [NFC] clang-formatting zos-alignment.c

fanbo-meng created this revision. fanbo-meng added a reviewer: abhina.sreeskantharajan. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D99514 Files:

[PATCH] D99508: [SystemZ][z/OS] Add test of leading zero length bitfield in const/volatile struct

This revision was automatically updated to reflect the committed changes. Closed by commit rGf1e0c7fdd720: [SystemZ][z/OS] Add test of leading zero length bitfield in const/volatile… (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D99508: [SystemZ][z/OS] Add test of leading zero length bitfield in const/volatile struct

fanbo-meng created this revision. fanbo-meng added a reviewer: abhina.sreeskantharajan. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D99508 Files:

[PATCH] D98864: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static variables on z/OS target

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0858f0e09e33: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static… (authored by fanbo-meng). Repository: rG LLVM Github

[PATCH] D98864: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static variables on z/OS target

fanbo-meng updated this revision to Diff 333737. fanbo-meng set the repository for this revision to rG LLVM Github Monorepo. fanbo-meng added a comment. rebasing to latest master Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98864/new/

[PATCH] D98890: [SystemZ][z/OS] Ignore leading zero width bitfield alignment on z/OS target

This revision was automatically updated to reflect the committed changes. Closed by commit rG6f91cf75d7f5: [SystemZ][z/OS] Ignore leading zero width bitfield alignment on z/OS target (authored by fanbo-meng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D98890: [SystemZ][z/OS] Ignore leading zero width bitfield alignment on z/OS target

fanbo-meng created this revision. fanbo-meng added reviewers: abhina.sreeskantharajan, hubert.reinterpretcast, Kai, SeanP, rsmith. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Zero length bitfield alignment is not respected

[PATCH] D98864: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static variables on z/OS target

fanbo-meng updated this revision to Diff 331559. fanbo-meng added a comment. remove accidentally included diff patch file CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98864/new/ https://reviews.llvm.org/D98864 Files: clang/include/clang/Basic/TargetInfo.h

[PATCH] D98864: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static variables on z/OS target

fanbo-meng updated this revision to Diff 331553. fanbo-meng added a comment. Herald added subscribers: sstefan1, phosek, s.egerton, jfb, mgrang, simoncook, fedor.sergeev, aheejin, krytarowski, dschuff. Herald added a reviewer: jfb. Herald added a reviewer: jdoerfert. clang-formatted CHANGES

[PATCH] D98864: [SystemZ][z/OS] Set maximum value to truncate attribute aligned to for static variables on z/OS target

fanbo-meng created this revision. fanbo-meng added reviewers: abhina.sreeskantharajan, muiez, Kai, anirudhp, DanielMcIntosh-IBM. fanbo-meng requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. On z/OS there is a hard limitation on on the

[PATCH] D89127: [SystemZ][z/OS] Update target specific __attribute__((aligned)) value for test

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd91234b21c1a: [SystemZ][z/OS] Update target specific __attribute__((aligned)) value for test (authored by fanbo-meng). Repository: rG LLVM Github

[PATCH] D89127: [SystemZ][z/OS] Update target specific __attribute__((aligned)) value for test

fanbo-meng created this revision. fanbo-meng added reviewers: abhina.sreeskantharajan, uweigand, Kai, hubert.reinterpretcast. Herald added subscribers: cfe-commits, krytarowski. Herald added a project: clang. fanbo-meng requested review of this revision. z/OS defaults to 16 bytes for

[PATCH] D88963: [SystemZ][z/OS] Add test of zero length bitfield type larger than zero length bitfield boundary

fanbo-meng created this revision. fanbo-meng added reviewers: abhina.sreeskantharajan, hubert.reinterpretcast. Herald added a project: clang. Herald added a subscriber: cfe-commits. fanbo-meng requested review of this revision. Repository: rG LLVM Github Monorepo

[PATCH] D88845: [SystemZ][z/OS] Set default alignment rules for z/OS target

fanbo-meng updated this revision to Diff 296269. fanbo-meng added a comment. fix no newline at end of file CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88845/new/ https://reviews.llvm.org/D88845 Files: clang/lib/Basic/Targets/OSTargets.h clang/test/CodeGen/zos-alignment.c Index:

[PATCH] D88845: [SystemZ][z/OS] Set default alignment rules for z/OS target

fanbo-meng created this revision. fanbo-meng added reviewers: abhina.sreeskantharajan, hubert.reinterpretcast, Kai, uweigand. Herald added a project: clang. Herald added a subscriber: cfe-commits. fanbo-meng requested review of this revision. Set the default alignment control variables for z/OS

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

fanbo-meng updated this revision to Diff 292259. fanbo-meng added a comment. remove trailing whitespace from diagnostic CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files: clang/include/clang/Basic/AlignedAllocation.h

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

fanbo-meng added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:7216 + "aligned %select{allocation|deallocation}0 function of type '%1' is " + "%select{only|not}4 available on %2 %select{%3 or newer|}4">; def

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

fanbo-meng updated this revision to Diff 292240. fanbo-meng set the repository for this revision to rG LLVM Github Monorepo. fanbo-meng added a comment. fix typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/

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

fanbo-meng updated this revision to Diff 292237. fanbo-meng added a comment. Herald added a reviewer: aaron.ballman. get platform name from triple instead of hardcoding, and convert its spelling CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611

[PATCH] D87624: [SystemZ][z/OS] Set default wchar_t type for zOS

fanbo-meng accepted this revision. fanbo-meng added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87624/new/ https://reviews.llvm.org/D87624

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

fanbo-meng updated this revision to Diff 291949. fanbo-meng added a comment. merge diagnostics CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files: clang/include/clang/Basic/AlignedAllocation.h

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

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<

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

fanbo-meng updated this revision to Diff 291638. fanbo-meng marked an inline comment as done. fanbo-meng added a comment. change vendor name from ibm to none for tests. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files:

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

fanbo-meng updated this revision to Diff 291616. fanbo-meng added a comment. be consistent with namespace qualification CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files: clang/include/clang/Basic/AlignedAllocation.h

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

fanbo-meng added inline comments. Comment at: clang/lib/Sema/SemaExprCXX.cpp:1841 -<< IsDelete << FD.getType().getAsString() << OSName -<< alignedAllocMinVersion(T.getOS()).getAsString(); Diag(Loc, diag::note_silence_aligned_allocation_unavailable);

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

fanbo-meng updated this revision to Diff 291593. fanbo-meng added a comment. Previous diff is incorrect and didn't include the entire change set, making a new one. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files:

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

fanbo-meng updated this revision to Diff 291590. fanbo-meng added a comment. emit the correct diagnostics for z/OS. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87611/new/ https://reviews.llvm.org/D87611 Files: clang/include/clang/Basic/AlignedAllocation.h

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

fanbo-meng created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. fanbo-meng requested review of this revision. Aligned allocation is not supported on z/OS. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D87611 Files:

[PATCH] D85722: [SystemZ][z/OS] enable trigraphs by default on z/OS

fanbo-meng requested changes to this revision. fanbo-meng added a comment. This revision now requires changes to proceed. Can you modify `clang/test/Frontend/trigraphs.cpp` and `clang/test/Lexer/cxx1z-trigraphs.cpp` ? These tests are assuming trigraphs are disabled by `-std=gnu++11