[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith updated this revision to Diff 300722. dexonsmith edited the summary of this revision. dexonsmith added a comment. Made the planned changes to SourceManager, wrapping the index in `LoadedSLocEntryIndex` which acts like an `Optional` (but still using 0 for a sentinel to allow zero-initi

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith planned changes to this revision. dexonsmith added a comment. I still need to investigate the LLDB test failures related to `ASTImporter`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89749/new/ https://reviews.llvm.org/D89749 ___

[PATCH] D90073: [Clang][CodeGen] fix failed assertion

nickdesaulniers updated this revision to Diff 300733. nickdesaulniers added a comment. - dyn_cast, else Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90073/new/ https://reviews.llvm.org/D90073 Files: clang/lib/CodeGen/CodeGenModule.cpp clang/t

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

DiggerLin marked 2 inline comments as done. DiggerLin added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames =

[PATCH] D90073: [Clang][CodeGen] fix failed assertion

erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land. Please make sure the commit message is significantly more descriptive here than what is on the review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revie

[PATCH] D89571: [clangd] Add textDocument/ast extension method to dump the AST

adamcz added inline comments. Comment at: clang-tools-extra/clangd/ClangdServer.cpp:795 +unsigned Start, End; +if (auto O = positionToOffset(Inputs->Inputs.Contents, R.start)) + Start = *O; Please rename O to anything else. *O below looks

[PATCH] D90063: [AIX] Also error on -G for link-only step

hubert.reinterpretcast accepted this revision. hubert.reinterpretcast added a comment. This revision is now accepted and ready to land. LGTM; thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90063/new/ https://reviews.llvm.org/D90063 ___

[PATCH] D90173: [PowerPC] Exploit splat instruction xxsplti32dx in Power10

Conanap created this revision. Conanap added reviewers: PowerPC, nemanjai, saghir. Conanap added projects: LLVM, clang, PowerPC. Herald added a subscriber: kbarton. Conanap requested review of this revision. Exploits the instruction xxsplti32dx. It can be used to materialize any 64 bit scalar/vec

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith planned changes to this revision. dexonsmith added a comment. In D89749#2354072 , @dexonsmith wrote: > I still need to investigate the LLDB test failures related to `ASTImporter`. (still to-do) CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith updated this revision to Diff 300737. dexonsmith added a comment. Added a unit test for `LoadedSLocEntryIndex`. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89749/new/ https://reviews.llvm.org/D89749 Files: clang/include/clang/Basic/SourceManager.h clang/lib/Basic/Sourc

[PATCH] D90101: [Clang][OpenMP][WIP] Avoid unnecessary privatization of mapper array when there is no user defined mapper

tianshilei1992 updated this revision to Diff 300738. tianshilei1992 added a comment. Fixed two test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90101/new/ https://reviews.llvm.org/D90101 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp cla

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

yaxunl created this revision. yaxunl added reviewers: tra, rjmccall. Herald added a subscriber: tpr. yaxunl requested review of this revision. Recently HIP toolchain made a change to use clang instead of opt/llc to do compilation (https://reviews.llvm.org/D81861). The intention is to make HIP too

[PATCH] D88645: [Annotation] Allows annotation to carry some additional constant arguments.

john.brawn added a comment. This also causes the AnnotateFunctions example to no longer build: /home/jb/work/llvm-project/clang/examples/AnnotateFunctions/AnnotateFunctions.cpp: In member function ‘virtual bool {anonymous}::AnnotateFunctionsConsumer::HandleTopLevelDecl(clang::DeclGroupRef)’:

[PATCH] D80531: [clang-tidy]: Added modernize-replace-disallow-copy-and-assign-macro

dblaikie added a comment. In D80531#2353268 , @njames93 wrote: > In D80531#2352944 , @kkleine wrote: > >> @dblaikie sorry for the late feedback. The `LLVM_ENABLE_WERROR:BOOL` will >> "Stop and fail the build, if a

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith updated this revision to Diff 300740. dexonsmith added a comment. Fix name of `LoadedSLocEntryIndices` (I documented it with this name, but was using `SLocEntryIndices` previously). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89749/new/ https://reviews.llvm.org/D89749 Fil

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith planned changes to this revision. dexonsmith added a comment. (still need to investigate ASTImporter) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89749/new/ https://reviews.llvm.org/D89749 ___ cfe-commits mailing list cfe-commits

[clang] 0387015 - SourceManager: Return non-const references in getOrCreateContentCache and related, NFC

Author: Duncan P. N. Exon Smith Date: 2020-10-26T14:07:46-04:00 New Revision: 0387015d7549fc2c0cc8c45d3e78114cc4fd153f URL: https://github.com/llvm/llvm-project/commit/0387015d7549fc2c0cc8c45d3e78114cc4fd153f DIFF: https://github.com/llvm/llvm-project/commit/0387015d7549fc2c0cc8c45d3e78114cc4fd

[PATCH] D90067: SourceManager: Return non-const references in getOrCreateContentCache and related, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rG0387015d7549: SourceManager: Return non-const references in getOrCreateContentCache and… (authored by dexonsmith). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

rjmccall added a comment. I have objections to the code change here. I'll leave the conceptual question to other people interested in the HIP toolchain. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90174/new/ https://reviews.llvm.org/D90174 __

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

yaxunl added a comment. In D90174#2354249 , @rjmccall wrote: > I have objections to the code change here. I'll leave the conceptual > question to other people interested in the HIP toolchain. Is it OK to introduce a clang codegen option e.g. -fp-contrac

[PATCH] D89608: Make sure Objective-C category support in IncludeSorter handles top-level imports

compositeprimes updated this revision to Diff 300749. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89608/new/ https://reviews.llvm.org/D89608 Files: clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp clang-tools-extra/unittests/clang-tidy/Inc

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

trixirt created this revision. trixirt added reviewers: alexfh, nickdesaulniers. trixirt added projects: clang, clang-tools-extra. Herald added subscribers: cfe-commits, xazax.hun, mgorny. trixirt requested review of this revision. Cleaning up -Wextra-semi-stmt in the linux kernel shows a high inc

[PATCH] D88987: [FPEnv][Clang][Driver] Use MarshallingInfoFlag for -fexperimental-strict-floating-point

kpn added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88987/new/ https://reviews.llvm.org/D88987 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/m

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

jasonliu added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames = CodeGenOpts.UniqueSectionNames;

[clang] c8f84bd - [Clang][CodeGen] fix failed assertion

Author: Nick Desaulniers Date: 2020-10-26T11:37:55-07:00 New Revision: c8f84bd0947d770a5df5e15d568a25a4f190df0b URL: https://github.com/llvm/llvm-project/commit/c8f84bd0947d770a5df5e15d568a25a4f190df0b DIFF: https://github.com/llvm/llvm-project/commit/c8f84bd0947d770a5df5e15d568a25a4f190df0b.di

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd marked 3 inline comments as done. compnerd added a comment. In D88859#2339159 , @martong wrote: > I value that you guys have a prototype on a fork that is being used in > production. But, upstreaming and these reviews give us the chance to evolve

[PATCH] D90073: [Clang][CodeGen] fix failed assertion

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc8f84bd0947d: [Clang][CodeGen] fix failed assertion (authored by nickdesaulniers). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

rnk accepted this revision. rnk added a comment. lgtm Please fix this case in a follow-up, though: static const signed int gv_signed = 0x; const signed int useit2() { return gv_signed; } I think your `APSInt` code only works on static data member constants. Repository: rG LLVM G

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

tra added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:486 + if (LangOpts.HIP) +Options.AllowFPOpFusion = llvm::FPOpFusion::Standard; + I don't think it's a good idea to force this. Perhaps a better way to address this would be to set HIP-

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

DiggerLin marked 2 inline comments as done. DiggerLin added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames =

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

DiggerLin marked an inline comment as done. DiggerLin added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames =

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

gribozavr2 added a comment. > I am having a hard time to accept "this is how it is implemented in our fork" > as a technical argument. Besides, I am not sure how could the Clang community > benefit about being backward compatible with a specialized fork and thus > making superfluous complicatio

[PATCH] D89899: [CodeGen] Implement [[likely]] and [[unlikely]] for the iteration statements

Mordante planned changes to this revision. Mordante marked 2 inline comments as done. Mordante added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:1854 + for(int i = 0; i != size; ++i) [[likely]] { +... // The loop the likely path of execut

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

yaxunl added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:486 + if (LangOpts.HIP) +Options.AllowFPOpFusion = llvm::FPOpFusion::Standard; + tra wrote: > I don't think it's a good idea to force this. > > Perhaps a better way to address this

[PATCH] D89844: [Clang][OpenMP] Fixed an issue of segment fault when using target nowait

tianshilei1992 updated this revision to Diff 300754. tianshilei1992 added a comment. Herald added a project: OpenMP. Herald added a subscriber: openmp-commits. Added a test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89844/new/ https://revie

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith added a comment. In D89749#2353602 , @teemperor wrote: > The tests are just loading a Clang module and then trying to import some > Decls into another ASTContext, so I don't think the error here is specific to > LLDB. We just don't have any AS

[clang] a2ac64d - Revert "Ensure that checkInitIsICE is called exactly once for every variable"

Author: Zequan Wu Date: 2020-10-26T11:59:55-07:00 New Revision: a2ac64dd905422ed84e273a98d846df022a5e2ec URL: https://github.com/llvm/llvm-project/commit/a2ac64dd905422ed84e273a98d846df022a5e2ec DIFF: https://github.com/llvm/llvm-project/commit/a2ac64dd905422ed84e273a98d846df022a5e2ec.diff LOG

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

jasonliu added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames = CodeGenOpts.UniqueSectionNames;

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

DiggerLin added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames = CodeGenOpts.UniqueSectionNames; ---

[clang] e56e7bd - Revert "Revert "Ensure that checkInitIsICE is called exactly once for every variable""

Author: Zequan Wu Date: 2020-10-26T12:08:57-07:00 New Revision: e56e7bd4698d29f6197a5f756b6f0bab02d0b3c8 URL: https://github.com/llvm/llvm-project/commit/e56e7bd4698d29f6197a5f756b6f0bab02d0b3c8 DIFF: https://github.com/llvm/llvm-project/commit/e56e7bd4698d29f6197a5f756b6f0bab02d0b3c8.diff LOG

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

tmroeder added inline comments. Comment at: clang-tools-extra/clang-tidy/linuxkernel/LinuxKernelTidyModule.cpp:25 "linuxkernel-must-check-errs"); +CheckFactories.registerCheck( +"linuxkernel-switch-semi"); Please fix the linter error, here a

[clang-tools-extra] 26750a1 - [clang-tidy] Silence unused variable warning in Release builds. NFCI.

Author: Benjamin Kramer Date: 2020-10-26T20:20:23+01:00 New Revision: 26750a1264b3df114a1efae7cde6f0784206b2ce URL: https://github.com/llvm/llvm-project/commit/26750a1264b3df114a1efae7cde6f0784206b2ce DIFF: https://github.com/llvm/llvm-project/commit/26750a1264b3df114a1efae7cde6f0784206b2ce.dif

[clang-tools-extra] 1e076a8 - Make sure Objective-C category support in IncludeSorter handles top-level imports

Author: Joe Turner Date: 2020-10-26T12:24:43-07:00 New Revision: 1e076a8d8099a6c179c8773f2e621f9408ee2402 URL: https://github.com/llvm/llvm-project/commit/1e076a8d8099a6c179c8773f2e621f9408ee2402 DIFF: https://github.com/llvm/llvm-project/commit/1e076a8d8099a6c179c8773f2e621f9408ee2402.diff LO

[PATCH] D89608: Make sure Objective-C category support in IncludeSorter handles top-level imports

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1e076a8d8099: Make sure Objective-C category support in IncludeSorter handles top-level… (authored by compositeprimes). Repository: rG LLVM Github

[clang] 22e6b18 - SourceManager: Fix an SLocEntry memory regression introduced with FileEntryRef

Author: Duncan P. N. Exon Smith Date: 2020-10-26T15:38:13-04:00 New Revision: 22e6b1863e74136908842d71b4f942313d89b273 URL: https://github.com/llvm/llvm-project/commit/22e6b1863e74136908842d71b4f942313d89b273 DIFF: https://github.com/llvm/llvm-project/commit/22e6b1863e74136908842d71b4f942313d89

[PATCH] D89580: SourceManager: Fix an SLocEntry memory regression introduced with FileEntryRef

This revision was automatically updated to reflect the committed changes. Closed by commit rG22e6b1863e74: SourceManager: Fix an SLocEntry memory regression introduced with FileEntryRef (authored by dexonsmith). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE L

[clang-tools-extra] d071bba - [clangd] Add back dependency on proto generated targets

Author: Kirill Bobyrev Date: 2020-10-26T20:39:10+01:00 New Revision: d071bba9a4477290d0b164a2d338fb22ad2d3c3d URL: https://github.com/llvm/llvm-project/commit/d071bba9a4477290d0b164a2d338fb22ad2d3c3d DIFF: https://github.com/llvm/llvm-project/commit/d071bba9a4477290d0b164a2d338fb22ad2d3c3d.diff

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

DiggerLin updated this revision to Diff 300766. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89986/new/ https://reviews.llvm.org/D89986 Files: clang/include/clang/Basic/CodeGenOptions.def clang/include/clang/Basic/LangOptions.def clang/lib/AS

[PATCH] D90187: [NFC] Remove max_align.c LIT testcase

Xiangling_L created this revision. Xiangling_L added reviewers: hubert.reinterpretcast, jasonliu, daltenty. Herald added a project: clang. Herald added a subscriber: cfe-commits. Xiangling_L requested review of this revision. Since we fixed the definition of `SuitableAlign` here: https://reviews.

[PATCH] D90188: Add support for attribute 'using_if_exists'

erik.pilkington created this revision. erik.pilkington added reviewers: rsmith, aaron.ballman, ldionne. Herald added subscribers: dexonsmith, ributzka, arphaman, jkorous. erik.pilkington requested review of this revision. This attribute applies to a using declaration, and permits importing a decl

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

jasonliu accepted this revision. jasonliu added a comment. This revision is now accepted and ready to land. LGTM with minor nit. Comment at: clang/lib/AST/Decl.cpp:1481 LinkageInfo LinkageComputer::getDeclLinkageAndVisibility(const NamedDecl *D) { - return getLVForDecl(D, -

[PATCH] D68997: Allow searching for prebuilt implicit modules.

arames updated this revision to Diff 300780. arames added a comment. Rebased on ToT. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68997/new/ https://reviews.llvm.org/D68997 Files: clang/docs/Modules.rst clang/include/clang/Driver/Options.td

Re: [clang] 76c0092 - Ensure that checkInitIsICE is called exactly once for every variable

Hi Richard, this makes clang assert when building chromium/win. https://bugs.chromium.org/p/chromium/issues/detail?id=1142009#c4 has a reduced repro. Could you take a look? Thanks, Nico On Mon, Oct 19, 2020 at 10:04 PM Richard Smith via cfe-commits < cfe-commits@lists.llvm.org> wrote: > > Autho

Re: [clang] 76c0092 - Ensure that checkInitIsICE is called exactly once for every variable

The original snippet: const int64_t& kGraceMs = AffiliationFetchThrottler::kGracePeriodAfterReconnectMs; ASSERT_NO_FATAL_FAILURE(AssertReleaseInBetween( true, kGraceMs * (1 - kPolicy.jitter_factor), kGraceMs)); kGracePeriod is declared here https://source.chromium.org/chromium/chr

[PATCH] D90063: [AIX] Also error on -G for link-only step

This revision was automatically updated to reflect the committed changes. Closed by commit rG3d4aebbb9d99: [AIX] Also error on -G for link-only step (authored by Xiangling_L). Changed prior to commit: https://reviews.llvm.org/D90063?vs=300391&id=300782#toc Repository: rG LLVM Github Monorepo

[clang] 3d4aebb - [AIX] Also error on -G for link-only step

Author: Xiangling Liao Date: 2020-10-26T16:51:28-04:00 New Revision: 3d4aebbb9d99642fcc277c8bd199ead400de2703 URL: https://github.com/llvm/llvm-project/commit/3d4aebbb9d99642fcc277c8bd199ead400de2703 DIFF: https://github.com/llvm/llvm-project/commit/3d4aebbb9d99642fcc277c8bd199ead400de2703.diff

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

sfertile added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:520 Options.DataSections = CodeGenOpts.DataSections; - Options.IgnoreXCOFFVisibility = CodeGenOpts.IgnoreXCOFFVisibility; Options.UniqueSectionNames = CodeGenOpts.UniqueSectionNames;

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

sfertile added a comment. Sorry, missed a comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89986/new/ https://reviews.llvm.org/D89986 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:/

[PATCH] D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility

sfertile added inline comments. Comment at: clang/test/CodeGen/aix-visibility-inlines-hidden.cpp:6 + +// RUN: %clang_cc1 -triple powerpc-unknown-aix -mcmodel=large -fvisibility-inlines-hidden -emit-llvm -o - -x c++ %s | \ +// RUN: FileCheck -check-prefixes=COMMON-IR,NOVISIBILIT

[PATCH] D90187: [NFC] Remove max_align.c LIT testcase

hubert.reinterpretcast accepted this revision. hubert.reinterpretcast 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/D90187/new/ https://reviews.llvm.org/D90187

[PATCH] D90079: Avoid unnecessary uses of `MDNode::getTemporary`, NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rGd4c667c9af03: Avoid unnecessary uses of `MDNode::getTemporary`, NFC (authored by dexonsmith). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://revi

[clang] 357715c - [NFC] Remove max_align.c LIT testcase

Author: Xiangling Liao Date: 2020-10-26T17:14:30-04:00 New Revision: 357715ce97d0bc937301b0b006d145c24a9d23aa URL: https://github.com/llvm/llvm-project/commit/357715ce97d0bc937301b0b006d145c24a9d23aa DIFF: https://github.com/llvm/llvm-project/commit/357715ce97d0bc937301b0b006d145c24a9d23aa.diff

[PATCH] D90187: [NFC] Remove max_align.c LIT testcase

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG357715ce97d0: [NFC] Remove max_align.c LIT testcase (authored by Xiangling_L). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION ht

[PATCH] D90192: WIP: Speculative fix for ASTImporter crash with SLocEntry changes

dexonsmith created this revision. dexonsmith added a reviewer: teemperor. Herald added subscribers: ributzka, martong. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. dexonsmith requested review of this revision. This is a speculative fix for the ASTImporter crash in https://

[libunwind] df6d2e8 - [libunwind] Add -Wno-dll-attribute-on-redeclaration when building for windows

Author: Martin Storsjö Date: 2020-10-26T23:23:01+02:00 New Revision: df6d2e8ab1a4212284e4763724a2211df2c7394a URL: https://github.com/llvm/llvm-project/commit/df6d2e8ab1a4212284e4763724a2211df2c7394a DIFF: https://github.com/llvm/llvm-project/commit/df6d2e8ab1a4212284e4763724a2211df2c7394a.diff

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith added a comment. @teemperor, I don't get test failures: % (cd build && env LIT_FILTER=Modules ninja -k20 check-lldb) ld: warning: spew Testing Time: 10.54s Excluded : 2105 Unsupported: 17 Passed : 13 Could this be because I used `-DLLDB_USE_SYSTEM_DEBUGS

[clang] ad1b9da - Prepend "__uniq" to symbol names hash with -funique-internal-linkage-names.

Author: Sriraman Tallam Date: 2020-10-26T14:24:28-07:00 New Revision: ad1b9daa4bf40c1907794fd5de7807aad1f0553c URL: https://github.com/llvm/llvm-project/commit/ad1b9daa4bf40c1907794fd5de7807aad1f0553c DIFF: https://github.com/llvm/llvm-project/commit/ad1b9daa4bf40c1907794fd5de7807aad1f0553c.dif

[PATCH] D89617: Prepend "uniq" to symbol names hash with -funique-internal-linkage-names

This revision was automatically updated to reflect the committed changes. Closed by commit rGad1b9daa4bf4: Prepend "__uniq" to symbol names hash with -funique-internal-linkage-names. (authored by tmsriram). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LL

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

dexonsmith updated this revision to Diff 300791. dexonsmith added a comment. Adding a test for multi-level indirection. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89488/new/ https://reviews.llvm.org/D89488 Files: clang/include/clang/Basic/FileManager.h clang/lib/Basic/FileManage

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

dexonsmith planned changes to this revision. dexonsmith added inline comments. Comment at: clang/lib/Basic/FileManager.cpp:219 + return FileEntryRef(*SeenFileInsertResult.first); +return FileEntryRef(*Value.V.get()); } arphaman wrote: > It looks like

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd marked 4 inline comments as done. compnerd added inline comments. Comment at: clang/lib/APINotes/APINotesYAMLCompiler.cpp:439 + static void enumeration(IO &IO, EnumExtensibilityKind &EEK) { +IO.enumCase(EEK, "none", EnumExtensibilityKind::None); +IO.enumCase(E

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd updated this revision to Diff 300796. compnerd added a comment. Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88859/new/ https://reviews.llvm.org/D88859 Files: clang/include/clang/APINotes/APINotesYAMLCompiler.h

[clang] c551ba0 - Run test only if X86 target is available

Author: Kiran Chandramohan Date: 2020-10-26T21:28:59Z New Revision: c551ba0e90bd2b49ef501d591f8362ba44e5484d URL: https://github.com/llvm/llvm-project/commit/c551ba0e90bd2b49ef501d591f8362ba44e5484d DIFF: https://github.com/llvm/llvm-project/commit/c551ba0e90bd2b49ef501d591f8362ba44e5484d.diff

[PATCH] D90009: [X86] VEX/EVEX prefix doesn't work for inline assembly.

kiranchandramohan added a comment. I have added a fix to run the test only when the X86 target is available. Please feel free to change if it is not the correct fix. https://github.com/llvm/llvm-project/commit/c551ba0e90bd2b49ef501d591f8362ba44e5484d Repository: rG LLVM Github Monorepo CHANG

[PATCH] D90194: [Driver] differentiate -stack-protector 0 from being unspecified

nickdesaulniers created this revision. nickdesaulniers added reviewers: void, manojgupta, rnk. Herald added subscribers: cfe-commits, dexonsmith, pengfei. Herald added a project: clang. nickdesaulniers requested review of this revision. Follow up to: 1. commit b7926ce6d7a8

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

rjmccall added a comment. Argh, sorry! I meant to say "I have *no* objections". CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90174/new/ https://reviews.llvm.org/D90174 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D90194: [Driver] differentiate -stack-protector 0 from being unspecified

nickdesaulniers added a comment. > An alternative approach to consider would be not adding a new enum, and > making -stack-protector ALWAYS be specified when invoking cc1. I did not > quantify the amount of changes to existing tests, but am happy to do so if > reviewers would like. Making `-st

[PATCH] D81748: [PowerPC] Add MMA builtin decoding and definitions

bsaleil updated this revision to Diff 300805. bsaleil added a comment. Herald added a subscriber: dexonsmith. Rebase patch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81748/new/ https://reviews.llvm.org/D81748 Files: clang/include/clang/AST/AS

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

njames93 added a comment. Is this not already handled by `-Wextra-semi`. If it isn't I feel like it should be handled there rather than in clang-tidy Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90180/new/ https://reviews.llvm.org/D90180 ___

[clang] 5159732 - [CodeView] Emit static data members as S_CONSTANTs.

Author: Amy Huang Date: 2020-10-26T15:30:35-07:00 New Revision: 515973222ed29abe49f241e89edb6854f44162d4 URL: https://github.com/llvm/llvm-project/commit/515973222ed29abe49f241e89edb6854f44162d4 DIFF: https://github.com/llvm/llvm-project/commit/515973222ed29abe49f241e89edb6854f44162d4.diff LOG

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG515973222ed2: [CodeView] Emit static data members as S_CONSTANTs. (authored by akhuang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

nickdesaulniers added a comment. In D90180#2354931 , @njames93 wrote: > Is this not already handled by `-Wextra-semi`. If it isn't I feel like it > should be handled there rather than in clang-tidy That was my initial thought too, but I think the value o

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

dexonsmith updated this revision to Diff 300811. dexonsmith added a comment. Updated to address remaining review comments. Note that we discussed double-redirection use case offline, and agreed it doesn't come up in practice (even with multiple RedirectingFileSystem layers) so we shouldn't supp

[PATCH] D89488: FileManager: Shrink FileEntryRef to the size of a pointer

dexonsmith marked 3 inline comments as done. dexonsmith added inline comments. Comment at: clang/lib/Basic/FileManager.cpp:217 FileEntry *FE; -if (LLVM_LIKELY(FE = Value.dyn_cast())) - return FileEntryRef(SeenFileInsertResult.first->first(), *FE); -return getFil

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

tra added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:486 + if (LangOpts.HIP) +Options.AllowFPOpFusion = llvm::FPOpFusion::Standard; + yaxunl wrote: > tra wrote: > > I don't think it's a good idea to force this. > > > > Perhaps a better w

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd updated this revision to Diff 300815. compnerd added a comment. Add more testing coverage Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88859/new/ https://reviews.llvm.org/D88859 Files: clang/include/clang/APINotes/APINotesYAMLCompiler.

Re: [clang] 76c0092 - Ensure that checkInitIsICE is called exactly once for every variable

Reduces to: int d; int &h = d; double e = h; ... but only fails under -fms-compatibility. On Mon, 26 Oct 2020 at 13:46, Nico Weber wrote: > The original snippet: > > const int64_t& kGraceMs = > AffiliationFetchThrottler::kGracePeriodAfterReconnectMs; > ASSERT_NO_FATAL_FAILURE(AssertR

[PATCH] D88859: APINotes: add APINotesYAMLCompiler

compnerd marked an inline comment as done. compnerd added inline comments. Comment at: clang/include/clang/APINotes/Types.h:25 +/// auditing. +enum class EnumExtensibilityKind { + None, martong wrote: > This seems a bit redundant to `Attrs.td`. I'd prefer to hav

[PATCH] D89844: [Clang][OpenMP] Fixed an issue of segment fault when using target nowait

jdoerfert accepted this revision. jdoerfert 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/D89844/new/ https://reviews.llvm.org/D89844 ___

[PATCH] D89802: [OpenMP] Add Passing in Original Declaration Names To Mapper API

jdoerfert added a comment. Looks generally good. We seem to have way to many places dealing with the mapping stuff but that is not your fault. One thing I found weird though: Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8078 + for (auto LI = C->component_lists_begin(

[PATCH] D89749: SourceManager: Don't allocate an SLocEntry until it's loaded

dexonsmith added a comment. In D89749#2354386 , @dexonsmith wrote: > That means it's not safe to store an address from `getSLocEntry` when there > will be another call. I can update `ASTImporter` to use a copy, but I think > this is too much of a gotcha.

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

trixirt updated this revision to Diff 300830. trixirt added a comment. Comment on the matcher Comment explaining need to look for empty macro Add test for normal switch Add test for switch with empty macro Fix formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://

Re: [clang] 76c0092 - Ensure that checkInitIsICE is called exactly once for every variable

Fixed in a5c7b46862ec0531964eb52329cdf009862abecf. On Mon, 26 Oct 2020 at 15:59, Richard Smith wrote: > Reduces to: > > int d; > int &h = d; > double e = h; > > ... but only fails under -fms-compatibility. > > On Mon, 26 Oct 2020 at 13:46, Nico Weber wrote: > >> The original snippet: >> >> co

[clang] a5c7b46 - Fix checking for C++98 ICEs in C++11-and-later mode to not consider use

Author: Richard Smith Date: 2020-10-26T16:59:48-07:00 New Revision: a5c7b46862ec0531964eb52329cdf009862abecf URL: https://github.com/llvm/llvm-project/commit/a5c7b46862ec0531964eb52329cdf009862abecf DIFF: https://github.com/llvm/llvm-project/commit/a5c7b46862ec0531964eb52329cdf009862abecf.diff

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

trixirt added a comment. Yes, this automates the fixit to something precise. There are about 10,000 problems in the kernel with -Wextra-semi, all of those fixes would be too many and diverse to practically submit. Also the -Wextra-semi fix removes just the semi and so it's fix will fail the kern

[PATCH] D90127: [clang][NFC] Rearrange Comment Token and Lexer fields to reduce padding

This revision was automatically updated to reflect the committed changes. Closed by commit rGb698ad00cbc7: [clang][NFC] Rearrange Comment Token and Lexer fields to reduce padding (authored by njames93). Changed prior to commit: https://reviews.llvm.org/D90127?vs=300623&id=300831#toc Repository

[clang] b698ad0 - [clang][NFC] Rearrange Comment Token and Lexer fields to reduce padding

Author: Nathan James Date: 2020-10-27T00:03:43Z New Revision: b698ad00cbc76f34f48bb639ffb1cfee47a9737e URL: https://github.com/llvm/llvm-project/commit/b698ad00cbc76f34f48bb639ffb1cfee47a9737e DIFF: https://github.com/llvm/llvm-project/commit/b698ad00cbc76f34f48bb639ffb1cfee47a9737e.diff LOG:

[PATCH] D89072: [CodeView] Emit static data members as S_CONSTANTs.

akhuang added a comment. Patch to emit signed ints for other S_CONSTANTs: https://reviews.llvm.org/D90199 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89072/new/ https://reviews.llvm.org/D89072 ___ cfe-

[PATCH] D90174: [HIP] Fix regressions due to fp contract change

yaxunl added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:486 + if (LangOpts.HIP) +Options.AllowFPOpFusion = llvm::FPOpFusion::Standard; + tra wrote: > yaxunl wrote: > > tra wrote: > > > I don't think it's a good idea to force this. > > >

[PATCH] D90180: [clang-tidy] find/fix unneeded semicolon after switch

njames93 added a comment. Taking a step back, clang-tidy checks are supposed to enforce guidelines for the specific module they live in. If there are 10'000 occurrences of a semi directly after a switch closing brace in the linux kernel code base it could be argued that its a style guideline of

<    1   2   3   >