[clang] 08388ad - [X86][AVX512FP16] Relax limitation to AVX512FP16 intrinsics. NFCI

2022-08-22 Thread Phoebe Wang via cfe-commits
Author: Phoebe Wang Date: 2022-08-23T11:35:16+08:00 New Revision: 08388ad81e9d65116b0857b0ebc9bcf7f2be5e31 URL: https://github.com/llvm/llvm-project/commit/08388ad81e9d65116b0857b0ebc9bcf7f2be5e31 DIFF: https://github.com/llvm/llvm-project/commit/08388ad81e9d65116b0857b0ebc9bcf7f2be5e31.diff L

[PATCH] D132342: [X86][AVX512FP16] Relax limitation to AVX512FP16 intrinsics. NFCI

2022-08-22 Thread Phoebe Wang via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG08388ad81e9d: [X86][AVX512FP16] Relax limitation to AVX512FP16 intrinsics. NFCI (authored by pengfei). Repository: rG LLVM Github Monorepo CHANGE

[PATCH] D132342: [X86][AVX512FP16] Relax limitation to AVX512FP16 intrinsics. NFCI

2022-08-22 Thread Phoebe Wang via Phabricator via cfe-commits
pengfei added a comment. Thanks @RKSimon Comment at: clang/lib/Headers/immintrin.h:223 +#if !(defined(_MSC_VER) || defined(__SCE__)) || __has_feature(modules) || \ +(defined(__AVX512VL__) && defined(__AVX512FP16__)) #include RKSimon wrote: > (style)

[PATCH] D132352: Introduce noread_thread_id to address the thread identification problem in coroutines

2022-08-22 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu updated this revision to Diff 454683. ChuanqiXu edited the summary of this revision. ChuanqiXu added a comment. Address comments: - Edit LangRef.rst. - Split the add-the-attributes part in later revisions. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132352/new/ https://revi

[clang] 2e36120 - [clang-format] Fix regressions in WhitespaceSensitiveMacros

2022-08-22 Thread via cfe-commits
Author: owenca Date: 2022-08-22T20:42:54-07:00 New Revision: 2e36120726ee46cbfd3eee1e9244bdd756dee92d URL: https://github.com/llvm/llvm-project/commit/2e36120726ee46cbfd3eee1e9244bdd756dee92d DIFF: https://github.com/llvm/llvm-project/commit/2e36120726ee46cbfd3eee1e9244bdd756dee92d.diff LOG: [

[PATCH] D132001: [clang-format] Fix regressions in WhitespaceSensitiveMacros

2022-08-22 Thread Owen Pan via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG2e36120726ee: [clang-format] Fix regressions in WhitespaceSensitiveMacros (authored by owenpan). Changed prior to commit: https://reviews.llvm.org/D132001?vs=453161&id=454685#toc Repository: rG LLVM

[PATCH] D132352: Introduce noread_thread_id to address the thread identification problem in coroutines

2022-08-22 Thread John McCall via Phabricator via cfe-commits
rjmccall added a comment. Okay. Doc parts LGTM, but I don't Comment at: llvm/include/llvm/IR/InstrTypes.h:1855 + } + void setNoReadThreadID() { addFnAttr(Attribute::NoReadThreadID); } + The closest existing precedent would suggest `doesNotReadThreadID` and

[clang] 47e7056 - [clang-format] Remove an extraneous test in TokenAnnotatorTest.cpp

2022-08-22 Thread via cfe-commits
Author: owenca Date: 2022-08-22T21:18:37-07:00 New Revision: 47e70569fc356c3dc1fbc444a55975dac586ce73 URL: https://github.com/llvm/llvm-project/commit/47e70569fc356c3dc1fbc444a55975dac586ce73 DIFF: https://github.com/llvm/llvm-project/commit/47e70569fc356c3dc1fbc444a55975dac586ce73.diff LOG: [

[PATCH] D132001: [clang-format] Fix regressions in WhitespaceSensitiveMacros

2022-08-22 Thread Jakub Kuderski via Phabricator via cfe-commits
kuhar added a comment. This seems to have caused a test failure: https://lab.llvm.org/buildbot/#/builders/109/builds/45138 Failed Tests (1): Clang-Unit :: Format/./FormatTests/TokenAnnotatorTest/UnderstandsLBracesInMacroDefinition Testing Time: 92.30s Skipped :38 Uns

[PATCH] D132001: [clang-format] Fix regressions in WhitespaceSensitiveMacros

2022-08-22 Thread Owen Pan via Phabricator via cfe-commits
owenpan added a comment. In D132001#3741467 , @kuhar wrote: > This seems to have caused a test failure: > https://lab.llvm.org/buildbot/#/builders/109/builds/45138 > > Failed Tests (1): > Clang-Unit :: > Format/./FormatTests/TokenAnnotatorTest/Unde

[PATCH] D131618: [WIP][Do NOT review] LLD related changes for -ffat-lto-objects support

2022-08-22 Thread Arda Unal via Phabricator via cfe-commits
arda marked 2 inline comments as done. arda added inline comments. Comment at: lld/ELF/InputFiles.cpp:1721 +Expected fatLTOData = IRObjectFile::findBitcodeInMemBuffer(mb); +if (!errorToBool(fatLTOData.takeError())) + return make(*fatLTOData, archiveName, offsetInArc

[clang] 6c6c4f6 - [CMake] Support passing arguments to build tool (bootstrap).

2022-08-22 Thread Carlos Alberto Enciso via cfe-commits
Author: Carlos Alberto Enciso Date: 2022-08-23T05:45:25+01:00 New Revision: 6c6c4f6a9b3ef2d7db937cb78784245ea8a61418 URL: https://github.com/llvm/llvm-project/commit/6c6c4f6a9b3ef2d7db937cb78784245ea8a61418 DIFF: https://github.com/llvm/llvm-project/commit/6c6c4f6a9b3ef2d7db937cb78784245ea8a614

[PATCH] D131665: [CMake] Support passing arguments to build tool (bootstrap).

2022-08-22 Thread Carlos Alberto Enciso via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG6c6c4f6a9b3e: [CMake] Support passing arguments to build tool (bootstrap). (authored by CarlosAlbertoEnciso). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1

[PATCH] D131665: [CMake] Support passing arguments to build tool (bootstrap).

2022-08-22 Thread Carlos Alberto Enciso via Phabricator via cfe-commits
CarlosAlbertoEnciso added a comment. In D131665#3740037 , @phosek wrote: > LGTM @phosek Thanks for your review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131665/new/ https://reviews.llvm.org/D131665

[PATCH] D132352: Introduce noread_thread_id to address the thread identification problem in coroutines

2022-08-22 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu updated this revision to Diff 454696. ChuanqiXu marked 3 inline comments as done. ChuanqiXu added a comment. Address comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132352/new/ https://reviews.llvm.org/D132352 Files: llvm/docs/LangRef.rst llvm/include/llvm/Bitcode

[PATCH] D132286: [clang][Interp] Implement function calls

2022-08-22 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder marked 4 inline comments as done. tbaeder added inline comments. Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:614 + +if (Optional T = classify(E->getType())) { + // Put arguments on the stack. aaron.ballman wrote: > Should this be using `

[PATCH] D132352: Introduce noread_thread_id to address the thread identification problem in coroutines

2022-08-22 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu updated this revision to Diff 454697. ChuanqiXu added a comment. I post the incorrect version the last time. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132352/new/ https://reviews.llvm.org/D132352 Files: llvm/docs/LangRef.rst llvm/include/llvm/Bitcode/LLVMBitCodes.h

[clang] 15b65bc - [Clang][LoongArch] Add initial LoongArch target and driver support

2022-08-22 Thread Weining Lu via cfe-commits
Author: Weining Lu Date: 2022-08-23T13:47:22+08:00 New Revision: 15b65bcd65195df8c9c0e84b64a1a825d9901ba6 URL: https://github.com/llvm/llvm-project/commit/15b65bcd65195df8c9c0e84b64a1a825d9901ba6 DIFF: https://github.com/llvm/llvm-project/commit/15b65bcd65195df8c9c0e84b64a1a825d9901ba6.diff LO

[PATCH] D130255: [Clang][LoongArch] Add initial LoongArch target and driver support

2022-08-22 Thread Lu Weining via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG15b65bcd6519: [Clang][LoongArch] Add initial LoongArch target and driver support (authored by SixWeining). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1302

[PATCH] D132434: Add noread_thread_id attribute to intrinsics

2022-08-22 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu created this revision. ChuanqiXu added reviewers: nhaehnle, jyknight, nikic, rjmccall, efriedma, fhahn, ychen, jdoerfert. ChuanqiXu added a project: clang. Herald added subscribers: jeroen.dobbelaere, wenlei, okura, kuter, arichardson. Herald added a project: All. ChuanqiXu requested rev

[PATCH] D132285: [LoongArch] Implement ABI lowering

2022-08-22 Thread Lu Weining via Phabricator via cfe-commits
SixWeining added a comment. In D132285#3739568 , @xry111 wrote: > In D132285#3737855 , @SixWeining > wrote: > >> Ignore zero-width bit fields and add a test. > > Thanks! There is a GCC test (https://gcc.gnu.or

[PATCH] D132285: [LoongArch] Implement ABI lowering

2022-08-22 Thread Lu Weining via Phabricator via cfe-commits
SixWeining updated this revision to Diff 454705. SixWeining added a comment. Add a test for struct with zero size arrays. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132285/new/ https://reviews.llvm.org/D132285 Files: clang/lib/Basic/Targets/L

<    1   2   3