[PATCH] D110215: [C++2a] [Module] Support extern C/C++ semantics

ChuanqiXu updated this revision to Diff 378415. ChuanqiXu added a comment. Format. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110215/new/ https://reviews.llvm.org/D110215 Files: clang/include/clang/Sema/Sema.h clang/lib/Sema/SemaDecl.cpp clang/lib/Sema/SemaModule.cpp

[PATCH] D111482: [Clang] [PowerPC] Fix header include typo in smmintrin.h

qiucf created this revision. qiucf added reviewers: jsji, nemanjai, PowerPC. Herald added subscribers: shchenz, kbarton. qiucf requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. It should be SSSE3 instead of SSE2. Repository: rG LLVM

[PATCH] D93298: [RISCV] add the MC layer support of Zfinx extension

achieveartificialintelligence added a comment. In D93298#3020917 , @frasercrmck wrote: > In D93298#3014160 , @jrtc27 wrote: > >> The amount of duplication here really depresses me and is only going to get >> worse

[PATCH] D107049: [clang-repl] Re-implement clang-interpreter as a test case.

v.g.vassilev added a comment. In D107049#3052557 , @leonardchan wrote: > Hi. I think our clang builders are failing from this after the reland >

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

hsmhsm updated this revision to Diff 378413. hsmhsm added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/ https://reviews.llvm.org/D111324 Files: clang/lib/CodeGen/CGExpr.cpp clang/lib/CodeGen/CodeGenFunction.h

[PATCH] D111316: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

hsmhsm updated this revision to Diff 378412. hsmhsm added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111316/new/ https://reviews.llvm.org/D111316 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp

[PATCH] D111293: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG04816829968c: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca() (authored by hsmhsm). Repository: rG LLVM Github Monorepo

[clang] 0481682 - [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

Author: hsmahesha Date: 2021-10-09T09:23:14+05:30 New Revision: 04816829968cf8aad0c71e8dc27a4a1e88c493a8 URL: https://github.com/llvm/llvm-project/commit/04816829968cf8aad0c71e8dc27a4a1e88c493a8 DIFF: https://github.com/llvm/llvm-project/commit/04816829968cf8aad0c71e8dc27a4a1e88c493a8.diff

[PATCH] D111293: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

hsmhsm updated this revision to Diff 378408. hsmhsm added a comment. Remove newly added test CodeGenObjC/static-alloc-init.m Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111293/new/ https://reviews.llvm.org/D111293 Files:

[PATCH] D111283: [clang] template / auto deduction deduces common sugar

mizvekov updated this revision to Diff 378406. mizvekov added a comment. - Tests and fix for dynamic exception spec corner case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111283/new/ https://reviews.llvm.org/D111283 Files:

[PATCH] D108441: [clang] Fix JSON AST output when a filter is used

xgupta added a comment. @woodruffw Please rebase the patch. clang/test/AST/ast-dump-if-json.cpp testcase is not applying cleanly. If you don't commit access, I can commit on your behalf. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 7eae8c6 - Don't update the vptr at the start of the destructor of a final class.

Author: Richard Smith Date: 2021-10-08T19:59:42-07:00 New Revision: 7eae8c6e62b2a91e71aade19324b9d2242bcb4ab URL: https://github.com/llvm/llvm-project/commit/7eae8c6e62b2a91e71aade19324b9d2242bcb4ab DIFF: https://github.com/llvm/llvm-project/commit/7eae8c6e62b2a91e71aade19324b9d2242bcb4ab.diff

[PATCH] D109948: [Clang] Enable _Complex __ibm128 type

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG85e565898ff3: [Clang] Enable _Complex __ibm128 type (authored by qiucf). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 85e5658 - [Clang] Enable _Complex __ibm128 type

Author: Qiu Chaofan Date: 2021-10-09T10:48:44+08:00 New Revision: 85e565898ff36e5e50ff6e7ed6bf82b1cfbf89d6 URL: https://github.com/llvm/llvm-project/commit/85e565898ff36e5e50ff6e7ed6bf82b1cfbf89d6 DIFF: https://github.com/llvm/llvm-project/commit/85e565898ff36e5e50ff6e7ed6bf82b1cfbf89d6.diff

[PATCH] D110215: [C++2a] [Module] Support extern C/C++ semantics

ChuanqiXu added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:9351 + (NewFD->isExternCContext() || NewFD->isExternCXXContext())) { +if (!getGlobalModule()) + Diag(NewFD->getLocation(), aaron.ballman wrote: > I'm a bit

[PATCH] D110215: [C++2a] [Module] Support extern C/C++ semantics

ChuanqiXu updated this revision to Diff 378401. ChuanqiXu added a comment. Address the comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110215/new/ https://reviews.llvm.org/D110215 Files: clang/include/clang/Sema/Sema.h clang/lib/Sema/SemaDecl.cpp

[PATCH] D111391: [NFC] [Clang] Use global enum for explicit float mode

This revision was automatically updated to reflect the committed changes. Closed by commit rG8a714722e2aa: [NFC] [Clang] Use global enum for explicit float mode (authored by qiucf). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111391/new/

[clang] 8a71472 - [NFC] [Clang] Use global enum for explicit float mode

Author: Qiu Chaofan Date: 2021-10-09T10:39:10+08:00 New Revision: 8a714722e2aa8a62d7812bd78babc99b7a6aeef6 URL: https://github.com/llvm/llvm-project/commit/8a714722e2aa8a62d7812bd78babc99b7a6aeef6 DIFF: https://github.com/llvm/llvm-project/commit/8a714722e2aa8a62d7812bd78babc99b7a6aeef6.diff

LLVM build master will be restarted soon

Hello, LLVM build master will be restarted at 9 PM PST today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D111475: [OpenMP] Add RTL function for getting number of threads in block.

This revision was automatically updated to reflect the committed changes. Closed by commit rGbad44d5f39bc: [OpenMP] Add RTL function for getting number of threads in block. (authored by jhuber6). Changed prior to commit: https://reviews.llvm.org/D111475?vs=378393=378397#toc Repository: rG

[clang] bad44d5 - [OpenMP] Add RTL function for getting number of threads in block.

Author: Joseph Huber Date: 2021-10-08T22:21:59-04:00 New Revision: bad44d5f39bc29c1062f0c4ce8b71720734ad3cb URL: https://github.com/llvm/llvm-project/commit/bad44d5f39bc29c1062f0c4ce8b71720734ad3cb DIFF: https://github.com/llvm/llvm-project/commit/bad44d5f39bc29c1062f0c4ce8b71720734ad3cb.diff

[PATCH] D109740: [OpenCL] Add atomic_half type builtins

haonanya added a comment. Hi, svenvh. Thanks for your review. Could you please help to submit the patch if there are no more inputs? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109740/new/ https://reviews.llvm.org/D109740 ___ cfe-commits

[PATCH] D111109: AddGlobalAnnotations for function with or without function body.

python3kgae added a comment. Updated summary. Tried to debug the linux fail with Hyper-V VM, cannot repro. Restarted Buildkite build to see if it still happens. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D09/new/

[clang] 222305d - PR51079: Treat thread_local variables with an incomplete class type as

Author: Richard Smith Date: 2021-10-08T18:46:01-07:00 New Revision: 222305d6ff6f6d156145d7de4f06d1c368383e41 URL: https://github.com/llvm/llvm-project/commit/222305d6ff6f6d156145d7de4f06d1c368383e41 DIFF: https://github.com/llvm/llvm-project/commit/222305d6ff6f6d156145d7de4f06d1c368383e41.diff

[PATCH] D111475: [OpenMP] Add RTL function for getting number of threads in block.

jhuber6 updated this revision to Diff 378393. jhuber6 added a comment. Fixing tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111475/new/ https://reviews.llvm.org/D111475 Files: clang/lib/CodeGen/CGOpenMPRuntimeAMDGCN.cpp

[PATCH] D110451: [IR] Increase max alignment to 4GB

aeubanks added a comment. Sorry about that, should be fixed with 20a0c482e0 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110451/new/ https://reviews.llvm.org/D110451

[PATCH] D100810: Use `GNUInstallDirs` to support custom installation dirs. -- LLVM

Ericson2314 updated this revision to Diff 378391. Ericson2314 added a comment. Rebase, avoid extra string ref conversions in llvm-config Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100810/new/ https://reviews.llvm.org/D100810 Files:

[PATCH] D110451: [IR] Increase max alignment to 4GB

leonardchan added a comment. Finally got the stacktrace: leonardchan@cp-snakewater:~/llvm-monorepo/llvm-build-1-clang-only-debug$ bin/clang++ "-cc1" "-triple" "x86_64-unknown-linux-gnu" "-emit-obj" "--mrelax-relocations" "-disable-free" "-main-file-name" "AArch64SpeculationHardening.cpp"

[PATCH] D111286: Add no_instrument_function attribute to Objective C methods as well

This revision was automatically updated to reflect the committed changes. Closed by commit rG0f00aa502d79: Add no_instrument_function attribute to Objective C methods as well (authored by hiraditya). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 0f00aa5 - Add no_instrument_function attribute to Objective C methods as well

Author: Aditya Kumar Date: 2021-10-08T17:54:44-07:00 New Revision: 0f00aa502d79acf8a92b0b8490489895a9f28829 URL: https://github.com/llvm/llvm-project/commit/0f00aa502d79acf8a92b0b8490489895a9f28829 DIFF: https://github.com/llvm/llvm-project/commit/0f00aa502d79acf8a92b0b8490489895a9f28829.diff

[clang] 04aff39 - Revert "Reland "[clang-repl] Re-implement clang-interpreter as a test case.""

Author: Leonard Chan Date: 2021-10-08T17:43:23-07:00 New Revision: 04aff395047ab39811eea1fd811a5874d3e4dfe4 URL: https://github.com/llvm/llvm-project/commit/04aff395047ab39811eea1fd811a5874d3e4dfe4 DIFF: https://github.com/llvm/llvm-project/commit/04aff395047ab39811eea1fd811a5874d3e4dfe4.diff

[PATCH] D111283: [clang] template / auto deduction deduces common sugar

mizvekov updated this revision to Diff 378388. mizvekov retitled this revision from "[clang] WIP: template / auto deduction deduces common sugar" to "[clang] template / auto deduction deduces common sugar". mizvekov edited the summary of this revision. mizvekov added a comment. - Add missing

[PATCH] D111477: DO NOT SUBMIT: workaround for context-sensitive use of non-type-template-parameter integer suffixes

dblaikie created this revision. dblaikie added reviewers: aprantl, probinson, JDevlieghere. dblaikie requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. There's a nuanced check about when to use suffixes on these integer

[PATCH] D111476: [modules] Make a module map referenced by a system map a system one too.

vsapsai added a comment. Used the mechanism simpler than the one with headers on purpose. // The #included file will be considered to be a system header if either

[PATCH] D110451: [IR] Increase max alignment to 4GB

aeubanks added a comment. I've comitted https://reviews.llvm.org/rGa0a493518245fa20fe15703a1be6babcb0c43c0a, if your stack trace is different, the fix will probably be very similar Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110451/new/

[PATCH] D111476: [modules] Make a module map referenced by a system map a system one too.

vsapsai created this revision. vsapsai added reviewers: Bigcheese, jansvoboda11. Herald added a subscriber: ributzka. vsapsai requested review of this revision. Herald added a project: clang. Mimic the behavior of including headers where a system includer makes an includee a system header too.

[PATCH] D111475: [OpenMP] Add RTL function for getting number of threads in block.

JonChesterfield accepted this revision. JonChesterfield added a comment. This revision is now accepted and ready to land. Great! I like the decrease in complexity of the compiler. Pretty close to dropping the nvptx and amdgpu subclasses entirely Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D111475: [OpenMP] Add RTL function for getting number of threads in block.

jhuber6 created this revision. jhuber6 added a reviewer: jdoerfert. Herald added subscribers: guansong, tpr, yaxunl, jvesely, jholewinski. jhuber6 requested review of this revision. Herald added subscribers: openmp-commits, cfe-commits, sstefan1. Herald added projects: clang, OpenMP. This patch

[PATCH] D111457: [clang][test] Add lit helper for windows paths

keith added inline comments. Comment at: clang/test/lit.cfg.py:60 +if platform.system() == 'Windows': +root_sep = 'C:\\' +else: rmaz wrote: > `c:\`? Do you mean lowercase or a single slash? I see ~2x the number of uppercase vs lowercase in this codebase

[PATCH] D111457: [clang][test] Add lit helper for windows paths

keith updated this revision to Diff 378380. keith added a comment. Herald added a subscriber: ormris. Fix tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111457/new/ https://reviews.llvm.org/D111457 Files:

[PATCH] D110451: [IR] Increase max alignment to 4GB

aeubanks added a comment. Yes a stack trace/repro would be nice. I see a couple places that might be causing issues, I'll try fixing those. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110451/new/ https://reviews.llvm.org/D110451

[PATCH] D80833: [CodeView] Add full repro to LF_BUILDINFO record

dexonsmith added a comment. If you do go with "off-by-default", the natural driver option to use would be the existing `-grecord-command-line`. You could use the same plumbing through `-cc1` that it does. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D94472: [clang][cli] Command line round-trip for HeaderSearch options

dexonsmith added a subscriber: aprantl. dexonsmith added a comment. In D94472#3052604 , @dexonsmith wrote: > Looking at `CommandLineArgs`, and its eventual forward to `MCTargetInfo`, the > existing design looks like a reference leak, which would also

[PATCH] D111468: [clang] Capture Framework when HeaderSearch is resolved via headermap

cishida created this revision. cishida added reviewers: dexonsmith, ributzka. cishida requested review of this revision. Herald added a project: clang. When building frameworks, headermaps responsible for mapping angle-included headers to their source file location are passed via `-I` and not

[PATCH] D110451: [IR] Increase max alignment to 4GB

leonardchan added a comment. Hi. It seems the reland caused an assertion to fail on our clang builders (https://luci-milo.appspot.com/ui/p/fuchsia/builders/prod/clang-linux-x64/b8834065059600721665/overview): [194/194] Building CXX object

[clang] 955dc34 - Fix TargetRegistry shlib build, clang edition

Author: Reid Kleckner Date: 2021-10-08T15:43:56-07:00 New Revision: 955dc3449acd64c0505a3823aa93a6af38b00782 URL: https://github.com/llvm/llvm-project/commit/955dc3449acd64c0505a3823aa93a6af38b00782 DIFF: https://github.com/llvm/llvm-project/commit/955dc3449acd64c0505a3823aa93a6af38b00782.diff

[PATCH] D111443: [Driver] Fix ToolChain::getSanitizerArgs

tra added a reviewer: eugenis. tra added a subscriber: eugenis. tra added a comment. In D111443#3052381 , @yaxunl wrote: > In D111443#3052099 , @tra wrote: > >> I'm curious why we need the cache at all. While the

[PATCH] D111454: Move TargetRegistry.(h|cpp) from Support to MC

lhames added a comment. Yeah -- this seems like a good idea to me. Thanks Reid! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111454/new/ https://reviews.llvm.org/D111454 ___ cfe-commits mailing list

[PATCH] D111454: Move TargetRegistry.(h|cpp) from Support to MC

MaskRay accepted this revision. MaskRay added a comment. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111454/new/ https://reviews.llvm.org/D111454 ___ cfe-commits mailing list

[PATCH] D111454: Move TargetRegistry.(h|cpp) from Support to MC

dexonsmith accepted this revision. dexonsmith 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/D111454/new/ https://reviews.llvm.org/D111454

[PATCH] D109707: [HIP] [AlwaysInliner] Disable AlwaysInliner to eliminate undefined symbols

gandhi21299 added a comment. Passed internal CI Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109707/new/ https://reviews.llvm.org/D109707 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D111454: Move TargetRegistry.(h|cpp) from Support to MC

rnk added a comment. Thanks for the review! I uploaded it a bit quickly because I wanted to kick off Bazel presubmit testing, since I haven't replicated that locally. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111454/new/

[PATCH] D111454: Move TargetRegistry.(h|cpp) from Support to MC

dexonsmith added a comment. Seems like a reasonable move to me. Comment at: llvm/include/llvm/MC/TargetRegistry.h:18-19 #ifndef LLVM_SUPPORT_TARGETREGISTRY_H #define LLVM_SUPPORT_TARGETREGISTRY_H Should be `LLVM_MC_TARGETREGISTRY_H` now.

[clang] 3db1ade - [Driver][test] Fix undefined-libs.cpp when CLANG_DEFAULT_UNWINDLIB is libunwind

Author: Fangrui Song Date: 2021-10-08T14:40:22-07:00 New Revision: 3db1ade368e538ba795486eea4923e331a8dde5a URL: https://github.com/llvm/llvm-project/commit/3db1ade368e538ba795486eea4923e331a8dde5a DIFF: https://github.com/llvm/llvm-project/commit/3db1ade368e538ba795486eea4923e331a8dde5a.diff

[clang] 64e9044 - Fix unintended fall-through.

Author: Richard Smith Date: 2021-10-08T14:25:17-07:00 New Revision: 64e9044ea43d2cee2a927bf221c5395a40a5e2fd URL: https://github.com/llvm/llvm-project/commit/64e9044ea43d2cee2a927bf221c5395a40a5e2fd DIFF: https://github.com/llvm/llvm-project/commit/64e9044ea43d2cee2a927bf221c5395a40a5e2fd.diff

[PATCH] D94472: [clang][cli] Command line round-trip for HeaderSearch options

dexonsmith added a comment. Looking at `CommandLineArgs`, and its eventual forward to `MCTargetInfo`, the existing design looks like a reference leak, which would also block freeing Clang memory when LLVM is still running. Given that 89ea0b05207d45c145fb525df554b3b986ae379b

[PATCH] D108592: [clang][Fuchsia] Support __attribute__((availability)) on Fuchsia

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/include/clang/Basic/LangOptions.def:432 +VALUE_LANGOPT(FuchsiaAPILevel, 32, 0, "Fuchsia API Level") + Comment

[PATCH] D111457: [clang][test] Add lit helper for windows paths

rmaz added inline comments. Comment at: clang/test/lit.cfg.py:60 +if platform.system() == 'Windows': +root_sep = 'C:\\' +else: `c:\`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111457/new/

[PATCH] D111242: Add `TypeLoc`-related matchers.

jcking1034 added a comment. @aaron.ballman that sounds like a good idea, what's the process for adding a release note? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111242/new/ https://reviews.llvm.org/D111242

[PATCH] D107049: [clang-repl] Re-implement clang-interpreter as a test case.

leonardchan added a comment. Hi. I think our clang builders are failing from this after the reland (https://luci-milo.appspot.com/ui/p/fuchsia/builders/toolchain.ci/clang-linux-x64/b8833962834812688769/overview): Script: --

[PATCH] D80833: [CodeView] Add full repro to LF_BUILDINFO record

dexonsmith added a comment. In D80833#2069411 , @aganea wrote: > In D80833#2069246 , @amccarth wrote: > >> Does embedding full paths affect distributed builds or build reproducibility? > > It does affect

[clang] 7ac308f - PR52073: Fix equivalence computation for lambda-expressions.

Author: Richard Smith Date: 2021-10-08T13:54:01-07:00 New Revision: 7ac308fb8f283249b89851702aa07f3ed9ac86d4 URL: https://github.com/llvm/llvm-project/commit/7ac308fb8f283249b89851702aa07f3ed9ac86d4 DIFF: https://github.com/llvm/llvm-project/commit/7ac308fb8f283249b89851702aa07f3ed9ac86d4.diff

[PATCH] D111382: Support _Float128 and F128 literal in C mode

MaskRay added a subscriber: zatrazz. MaskRay added a comment. This is to make more glibc files compilable with Clang. (@zatrazz) In D111382#3052514 , @hubert.reinterpretcast wrote: > Are we sure this is wise? We know that any future standard C++ type

[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

noajshu marked 3 inline comments as not done. noajshu added inline comments. Comment at: llvm/lib/Support/Caching.cpp:36 + SmallString<10> CacheName = CacheNameRef; return [=](unsigned Task, StringRef Key) -> AddStreamFn { noajshu wrote: > tejohnson wrote:

[clang] 05d46f6 - [NFC][sanitizer] Remove sanitizer_persistent_allocator.cpp

Author: Vitaly Buka Date: 2021-10-08T13:43:28-07:00 New Revision: 05d46f627c49ca9a576150be910a869034ced764 URL: https://github.com/llvm/llvm-project/commit/05d46f627c49ca9a576150be910a869034ced764 DIFF: https://github.com/llvm/llvm-project/commit/05d46f627c49ca9a576150be910a869034ced764.diff

[PATCH] D111382: Support _Float128 and F128 literal in C mode

hubert.reinterpretcast added a comment. Are we sure this is wise? We know that any future standard C++ type can't be the same as `__float128` (without changing the latter) because `__float128` mangles the same as 128-bit `long double`. So by making `__float128` the same as `_Float128` in C,

[PATCH] D111453: Delay trailing constraing evaluation so we short-circuit

erichkeane abandoned this revision. erichkeane added a comment. Looking into my other issue, I think I'm reasonably convinced this is wrong. I was looking into the PR44833 blocker for 'ranges' which hits the VisitCXXRecordMethod part, and removing THAT breaks the constraint entirely, so I

[PATCH] D111463: [OpenMP] Remove doing assumption propagation in the front end.

jhuber6 created this revision. jhuber6 added a reviewer: jdoerfert. Herald added subscribers: guansong, yaxunl. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. This patch removes the assumption propagation that was added

[PATCH] D102488: [clang][deps] Prune unused header search paths

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102488/new/ https://reviews.llvm.org/D102488

[PATCH] D111462: [Driver][OpenBSD] Use ToolChain reference instead of getToolChain().

fcambus created this revision. fcambus added a reviewer: brad. Herald added a subscriber: fedor.sergeev. fcambus 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/D111462

[PATCH] D102923: [clang][lex] Remark on search path usage

dexonsmith added inline comments. Comment at: clang/test/Preprocessor/header-search-user-entries.c:9-11 +// CHECK: remark: user-provided search path used: '{{.*}}/header-search-user-entries/a' +// CHECK: remark: user-provided search path used:

[PATCH] D111453: Delay trailing constraing evaluation so we short-circuit

erichkeane added a comment. So debugging a DIFFERENT issue, I found that this is likely incomplete since we likely have a similar problem in `VisitCXXMethodDecl` below this. I can extend the tests here/fix easily enough, if we decide this is the right way around this. CHANGES SINCE LAST

[PATCH] D110935: [NFC] Update vec_extract builtin signatures to take signed int.

This revision was automatically updated to reflect the committed changes. Closed by commit rG03bfddae5080: [NFC] Update vec_extract builtin signatures to take signed int. (authored by amyk). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110935/new/

[clang] 03bfdda - [NFC] Update vec_extract builtin signatures to take signed int.

Author: Amy Kwan Date: 2021-10-08T15:09:53-05:00 New Revision: 03bfddae5080f8b92d86342d68439c7f07838369 URL: https://github.com/llvm/llvm-project/commit/03bfddae5080f8b92d86342d68439c7f07838369 DIFF: https://github.com/llvm/llvm-project/commit/03bfddae5080f8b92d86342d68439c7f07838369.diff

[PATCH] D102923: [clang][lex] Remark on search path usage

dexonsmith added inline comments. Comment at: clang/test/Preprocessor/header-search-user-entries.c:9-11 +// CHECK: remark: user-provided search path used: '{{.*}}/header-search-user-entries/a' +// CHECK: remark: user-provided search path used:

[clang] e1a9c04 - [test] Fixup builtin-assume-aligned.c

Author: Arthur Eubanks Date: 2021-10-08T13:07:28-07:00 New Revision: e1a9c0443533723300f76db10dcb221733420882 URL: https://github.com/llvm/llvm-project/commit/e1a9c0443533723300f76db10dcb221733420882 DIFF:

[PATCH] D111443: [Driver] Fix ToolChain::getSanitizerArgs

yaxunl added a comment. In D111443#3052099 , @tra wrote: > I'm curious why we need the cache at all. While the construction of sanitizer > args is hairy, it's only called few times from the driver and will be lost in > the noise compared to everything

[PATCH] D111367: [Driver] Change -dumpmachine to respect --target/LLVM_DEFAULT_TARGET_TRIPLE verbatim

MaskRay added a comment. Agree that Red Hat GCC switching to `x86_64-redhat-linux-gnu` is the best long-term approach. > (Edit: @tstellar's approach to make LLVM get a x86_64-redhat-linux-gnu triple > avoids that problem, so that sounds good to me.) Let the `config.guess` removal patch

[PATCH] D111242: Add `TypeLoc`-related matchers.

aaron.ballman added a comment. Oh! I did think of something else we should do -- can you add a release note to make users aware of the awesome new functionality? It doesn't have to be super detailed, but enough to alert people to the fact that this is significant new functionality.

[PATCH] D111459: Fix a diagnoses-valid in C++20 with variadic macros

aaron.ballman created this revision. aaron.ballman added reviewers: rsmith, erichkeane, cor3ntin. aaron.ballman requested review of this revision. Herald added a project: clang. C++20 and later allow you to pass no argument for the ... parameter in a variadic macro, whereas earlier language modes

[PATCH] D109632: [clang] de-duplicate methods from AST files

rmaz added a comment. In D109632#3046976 , @vsapsai wrote: > These are interesting results. I'm curious to measure the time spent in > `ASTReader::ReadMethodPool`. Updated numbers from today, looks like we added a lot more modular deps since last

[PATCH] D111400: [Clang] Implement P2242R3

hubert.reinterpretcast added a comment. I am concerned that the general direction of this patch (to allow the previously ill-formed constexpr functions as an extension under older language levels) is based upon bugs/divergences-from-orthodoxy in Clang's implementation of C++20 (noting that the

[PATCH] D109948: [Clang] Enable _Complex __ibm128 type

rjmccall accepted this revision. rjmccall 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/D109948/new/ https://reviews.llvm.org/D109948

[PATCH] D111324: [CFE][Codegen] Remove CodeGenFunction::InitTempAlloca()

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Thanks. This LGTM when all the patches it depends on are checked in. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111324/new/

[PATCH] D109950: [Clang] Enable IC/IF mode for __ibm128

rjmccall added a comment. I agree. It doesn't have to be a CodeGen test, just anything that directly verifies that we get the right type, since I think those calls can succeed due to promotion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D111457: [clang][test] Add lit helper for windows paths

keith created this revision. keith added reviewers: jroelofs, rmaz. keith requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This adds 2 new lit helpers (only to the clang config) `%{rootsep}` and `%{sep}`, these allow writing tests that

[PATCH] D111453: Delay trailing constraing evaluation so we short-circuit

erichkeane added a comment. The implementation seems to have tried checking this constraint 2x, and the first time required fully instantiating the constraint. The only condition in the check is PartialOrdering here:

[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

noajshu updated this revision to Diff 378328. noajshu marked 2 inline comments as done. noajshu added a comment. Add documentation for new parameters to localCache. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111371/new/

[PATCH] D111453: Delay trailing constraing evaluation so we short-circuit

erichkeane created this revision. erichkeane added reviewers: rsmith, hubert.reinterpretcast, saar.raz, arthur.j.odwyer, aaron.ballman, changyu, nwilson, cor3ntin. erichkeane requested review of this revision. As reported as a part of PR47508 (and somewhat confirmed in PR44833), our behavior to

[PATCH] D108592: [clang][Fuchsia] Support __attribute__((availability)) on Fuchsia

haowei updated this revision to Diff 378324. haowei added a comment. Added unit test when "-ffuchsia-api-level" is supplied with a non integer value. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108592/new/ https://reviews.llvm.org/D108592

[clang] a6891d2 - [clang] Set max allowed alignment to 2^32

Author: Arthur Eubanks Date: 2021-10-08T11:44:15-07:00 New Revision: a6891d2104fc2c491885aece5c9d040926ff1ef2 URL: https://github.com/llvm/llvm-project/commit/a6891d2104fc2c491885aece5c9d040926ff1ef2 DIFF:

[PATCH] D111250: [clang] Set max allowed alignment to 2^32

This revision was automatically updated to reflect the committed changes. Closed by commit rGa6891d2104fc: [clang] Set max allowed alignment to 2^32 (authored by aeubanks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111250/new/

[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

noajshu marked 3 inline comments as done. noajshu added inline comments. Comment at: llvm/lib/Support/Caching.cpp:36 + SmallString<10> CacheName = CacheNameRef; return [=](unsigned Task, StringRef Key) -> AddStreamFn { tejohnson wrote: > Is this copy

[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

noajshu updated this revision to Diff 378316. noajshu added a comment. Adjust cache name/prefix customization. Move customization arguments to be first. Convert to twines, remove unnecessary copy. Separate customized file prefix from customized cache name (to avoid chaning any behavior from

[clang] b0f6879 - [clang] Option control afn flag

Author: Masoud Ataei Date: 2021-10-08T14:26:14-04:00 New Revision: b0f68791f0ad867118497daf6aa23f9a71500012 URL: https://github.com/llvm/llvm-project/commit/b0f68791f0ad867118497daf6aa23f9a71500012 DIFF: https://github.com/llvm/llvm-project/commit/b0f68791f0ad867118497daf6aa23f9a71500012.diff

[PATCH] D111443: [Driver] Fix ToolChain::getSanitizerArgs

tra added a comment. I'm curious why we need the cache at all. While the construction of sanitizer args is hairy, it's only called few times from the driver and will be lost in the noise compared to everything else. Comment at: clang/include/clang/Driver/ToolChain.h:165 -

[PATCH] D107049: [clang-repl] Re-implement clang-interpreter as a test case.

v.g.vassilev added inline comments. Comment at: clang/unittests/Interpreter/CMakeLists.txt:5 + OrcJIT + Support ) thakis wrote: > Why are these additions needed here again? This change doesn't add any code > to ClangReplInterpreterTests as far as I can

[PATCH] D111283: [clang] WIP: template / auto deduction deduces common sugar

mizvekov updated this revision to Diff 378313. mizvekov edited the summary of this revision. mizvekov added a comment. - Handle all type classes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111283/new/ https://reviews.llvm.org/D111283 Files:

[PATCH] D94472: [clang][cli] Command line round-trip for HeaderSearch options

saudi added inline comments. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:677 + SwapOpts(Res); + bool Success2 = Parse(Res, GeneratedArgs1, Diags); + jansvoboda11 wrote: > saudi wrote: > > jansvoboda11 wrote: > > > saudi wrote: > > > > Hello, > > > >

[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

tejohnson added inline comments. Comment at: llvm/include/llvm/Support/Caching.h:68 +/// already exist. +Expected localCache(StringRef CacheDirectoryPath, + AddBufferFn AddBuffer, Nit: document new parameter.

  1   2   3   >