[clang] 4857b6f - Revert "Store OptTable::Info::Name as a StringRef"

Author: Douglas Yung Date: 2022-12-06T19:37:05-08:00 New Revision: 4857b6f8fff614c89ed35031c298df81394459af URL: https://github.com/llvm/llvm-project/commit/4857b6f8fff614c89ed35031c298df81394459af DIFF: https://github.com/llvm/llvm-project/commit/4857b6f8fff614c89ed35031c298df81394459af.diff

r291491 - Fixing test to work when the compiler defaults to a different C++ standard version.

Author: dyung Date: Mon Jan 9 16:20:10 2017 New Revision: 291491 URL: http://llvm.org/viewvc/llvm-project?rev=291491&view=rev Log: Fixing test to work when the compiler defaults to a different C++ standard version. Differential Revision: https://reviews.llvm.org/D28418 Modified: cfe/trunk

r279603 - Adding an additional test to ensure the frame pointer is emitted

Author: dyung Date: Tue Aug 23 21:02:32 2016 New Revision: 279603 URL: http://llvm.org/viewvc/llvm-project?rev=279603&view=rev Log: Adding an additional test to ensure the frame pointer is emitted when compiling with optimization when PS4 is the target. Modified: cfe/trunk/test/Driver/frame-

r297799 - Fix test on Windows. Only a single backslash was required, not a double.

Author: dyung Date: Tue Mar 14 20:38:24 2017 New Revision: 297799 URL: http://llvm.org/viewvc/llvm-project?rev=297799&view=rev Log: Fix test on Windows. Only a single backslash was required, not a double. Modified: cfe/trunk/test/Modules/modules-cache-path-canonicalization.m Modified: cfe/tr

r294534 - Add check that "#define unix 1" is present for the PS4 target.

Author: dyung Date: Wed Feb 8 18:07:38 2017 New Revision: 294534 URL: http://llvm.org/viewvc/llvm-project?rev=294534&view=rev Log: Add check that "#define unix 1" is present for the PS4 target. Modified: cfe/trunk/test/Preprocessor/init.c Modified: cfe/trunk/test/Preprocessor/init.c URL:

r322417 - Fix test on Windows that was added in r322382.

Author: dyung Date: Fri Jan 12 14:32:01 2018 New Revision: 322417 URL: http://llvm.org/viewvc/llvm-project?rev=322417&view=rev Log: Fix test on Windows that was added in r322382. The test was using "%clang++" which on Windows became "clang.exe++". Use %clangxx instead. Reviewed by Paul Robinson

r322778 - [DOXYGEN] Fix doxygen and content issues in xmmintrin.h

Author: dyung Date: Wed Jan 17 14:53:15 2018 New Revision: 322778 URL: http://llvm.org/viewvc/llvm-project?rev=322778&view=rev Log: [DOXYGEN] Fix doxygen and content issues in xmmintrin.h - Fix inaccurate instruction listings. - Fix small issues in _mm_getcsr and _mm_setcsr. - Fix description of

r323285 - Fix test Driver/solaris-ld.c for Windows.

Author: dyung Date: Tue Jan 23 16:05:01 2018 New Revision: 323285 URL: http://llvm.org/viewvc/llvm-project?rev=323285&view=rev Log: Fix test Driver/solaris-ld.c for Windows. - Test needs to be able to handle "clang.exe" on Windows - Test needs to be able to handle either '/' or '\\' used as the p

r296066 - Recently a change was made to this test in r294639 which fails when the

Author: dyung Date: Thu Feb 23 19:25:02 2017 New Revision: 296066 URL: http://llvm.org/viewvc/llvm-project?rev=296066&view=rev Log: Recently a change was made to this test in r294639 which fails when the compiler is run in a mode where the default C++ standard is newer than C++03. The reason is be

[PATCH] D26827: Make tests more robust with Release builds

dyung created this revision. dyung added a reviewer: ABataev. dyung added a subscriber: cfe-commits. These two tests in many places looks for a pattern: ... br ... label % ... ... In non-release builds of the compiler, this works fine. For example: br label %omp.inner.for.cond, !dbg !313 omp.i

[PATCH] D26827: Make tests more robust with Release builds

This revision was automatically updated to reflect the committed changes. Closed by commit rL287851: Make these tests work more reliably with Release builds. (authored by dyung). Changed prior to commit: https://reviews.llvm.org/D26827?vs=78449&id=79181#toc Repository: rL LLVM https://revie

r287851 - Make these tests work more reliably with Release builds.

Author: dyung Date: Wed Nov 23 19:53:38 2016 New Revision: 287851 URL: http://llvm.org/viewvc/llvm-project?rev=287851&view=rev Log: Make these tests work more reliably with Release builds. Differential Revision: https://reviews.llvm.org/D26827 Modified: cfe/trunk/test/OpenMP/for_simd_codegen

r289075 - Fixing test to work when the compiler defaults to a different C++ standard version.

Author: dyung Date: Thu Dec 8 11:27:20 2016 New Revision: 289075 URL: http://llvm.org/viewvc/llvm-project?rev=289075&view=rev Log: Fixing test to work when the compiler defaults to a different C++ standard version. Differential Revision: https://reviews.llvm.org/D27488 Modified: cfe/trunk

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung added a comment. ping http://reviews.llvm.org/D19048 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung added a comment. In http://reviews.llvm.org/D19048#414568, @aprantl wrote: > Thanks! > > Is there anything meaningful that could be CHECKed here? > Usually a crash means that we previously had a code path without test > coverage. This was definitely a code path without test coverage. W

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung updated this revision to Diff 55457. dyung added a comment. Modified the test to emit llvm assembly and check for the presence of a DIModule in the output. http://reviews.llvm.org/D19048 Files: test/Modules/Inputs/getSourceDescriptor-crash/h1.h test/Modules/Inputs/getSourceDescriptor

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung updated this revision to Diff 55460. dyung added a comment. Change test to not use the driver. http://reviews.llvm.org/D19048 Files: test/Modules/Inputs/getSourceDescriptor-crash/h1.h test/Modules/Inputs/getSourceDescriptor-crash/module.modulemap test/Modules/getSourceDescriptor-cra

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung added inline comments. Comment at: test/Modules/getSourceDescriptor-crash.cpp:6 @@ +5,2 @@ + +// CHECK: DIModule2 aprantl wrote: > This doesn't actually appear in the output :-) > You probably want to at least check for the correct name of the module and >

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung updated this revision to Diff 55495. dyung added a comment. Fixed up test and check for that DIImportedEntity and DIModule are present in the generated output. http://reviews.llvm.org/D19048 Files: test/Modules/Inputs/getSourceDescriptor-crash/h1.h test/Modules/Inputs/getSourceDescri

r268344 - Adding a test for a compiler crash that was fixed in r248069.

Author: dyung Date: Mon May 2 19:29:56 2016 New Revision: 268344 URL: http://llvm.org/viewvc/llvm-project?rev=268344&view=rev Log: Adding a test for a compiler crash that was fixed in r248069. Differential Revision: http://reviews.llvm.org/D19048 Added: cfe/trunk/test/Modules/Inputs/getSou

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

This revision was automatically updated to reflect the committed changes. Closed by commit rL268344: Adding a test for a compiler crash that was fixed in r248069. (authored by dyung). Changed prior to commit: http://reviews.llvm.org/D19048?vs=55495&id=55925#toc Repository: rL LLVM http://re

Re: [PATCH] D18708: Set C99 as default C Standard for PS4 target

dyung added a comment. In http://reviews.llvm.org/D18708#390132, @rsmith wrote: > What's the motivation for this change? Clang's C11 mode should accept all > code accepted by its C99 mode (and conversely, most or perhaps all of the C11 > language features are accepted by default in C99 mode as

Re: [PATCH] D18708: Set C99 as default C Standard for PS4 target

dyung added a comment. In http://reviews.llvm.org/D18708#390166, @rsmith wrote: > In http://reviews.llvm.org/D18708#390150, @dyung wrote: > > > From my understanding, there are 2 issues that block us. The first is that > > we currently do not ship all of the header files for C11. The second is

Re: [PATCH] D18708: Set C99 as default C Standard for PS4 target

dyung updated this revision to Diff 52622. dyung added a comment. Moved the test to test/Preprocessor as suggested by Gao and found a nice place to add it. http://reviews.llvm.org/D18708 Files: include/clang/Frontend/CompilerInvocation.h lib/Frontend/CompilerInvocation.cpp test/Preproces

[PATCH] D19003: Set C99 as default C Standard for PS4 target

dyung created this revision. dyung added reviewers: rsmith, ygao. dyung added a subscriber: cfe-commits. On the PS4, the default C standard is C99 which differs from the current default of C11. This patch makes the default C99 when targeting the PS4. This change includes the changes previously p

[PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung created this revision. dyung added a reviewer: aprantl. dyung added a subscriber: cfe-commits. In our internal testing, we found a crash in the compiler which was reduced to the attached test case. The issue was "fixed" a while back with Adrian's change in r248069 as the code causing the c

Re: [PATCH] D19003: Set C99 as default C Standard for PS4 target

dyung added a comment. Ping http://reviews.llvm.org/D19003 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19048: Test for a module related crash in CGDebugInfo.cpp

dyung added a comment. Any thoughts on this test? http://reviews.llvm.org/D19048 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D19003: Set C99 as default C Standard for PS4 target

dyung updated this revision to Diff 54956. dyung added a comment. Updated diff to fix up the comments based on Paul's feedback. Also tightened up one of the #if blocks to not include something that did not require C11. http://reviews.llvm.org/D19003 Files: include/clang/Frontend/CompilerInvo

[clang-tools-extra] f85b153 - Add -fexceptions to test as it uses them and fails on platforms where it is not on by default (like the PS4).

Author: Douglas Yung Date: 2021-01-14T13:27:06-08:00 New Revision: f85b1531666d12e6ca9b29217b5c5da30dfd93cf URL: https://github.com/llvm/llvm-project/commit/f85b1531666d12e6ca9b29217b5c5da30dfd93cf DIFF: https://github.com/llvm/llvm-project/commit/f85b1531666d12e6ca9b29217b5c5da30dfd93cf.diff

[clang] be68c92 - [NFC] Add -std=c11 to attr-availability.c

Author: Douglas Yung Date: 2021-01-15T21:05:49-08:00 New Revision: be68c9222b85815612e6bd8bc606a87e3111a0fb URL: https://github.com/llvm/llvm-project/commit/be68c9222b85815612e6bd8bc606a87e3111a0fb DIFF: https://github.com/llvm/llvm-project/commit/be68c9222b85815612e6bd8bc606a87e3111a0fb.diff

[clang] 323a6bf - Add "REQUIRES: arm-registered-target" line to test added in D108603.

Author: Douglas Yung Date: 2021-08-24T22:22:16-07:00 New Revision: 323a6bfbb8cf8082e22dc482abeaf6664d84bbdf URL: https://github.com/llvm/llvm-project/commit/323a6bfbb8cf8082e22dc482abeaf6664d84bbdf DIFF: https://github.com/llvm/llvm-project/commit/323a6bfbb8cf8082e22dc482abeaf6664d84bbdf.diff

<    1   2