[PATCH] D108367: [NFC] computeSPIRKernelABIInfo(): use SPIRABInfo

pekka.jaaskelainen added a comment. In D108367#2966746 , @pekka.jaaskelainen wrote: > @Anastasia I used to have svn commit access, but haven't committed anything > for a long while in LLVM, so don't know if I can get them back to github by > just

[PATCH] D109681: [RISCV] add Half-precision test for vle/vse

achieveartificialintelligence created this revision. achieveartificialintelligence added reviewers: craig.topper, HsiangKai. Herald added subscribers: vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck,

[PATCH] D105169: [Clang/Test]: Rename enable_noundef_analysis to disable-noundef-analysis and turn it off by default

hyeongyukim added a comment. I did some experiments to confirm the benefits of adding a noundef attribute to the function parameter. (test result link ) One of the most significant advantages of

[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

craig.topper added a comment. In D109658#2996714 , @pengfei wrote: > In D109658#2996412 , @craig.topper > wrote: > >> Does gcc use the same builtin name? Our general policy is to have the same >> interface as

[PATCH] D108621: [HIPSPV] Add CUDA->SPIR-V address space mapping

bader added inline comments. Comment at: clang/lib/Basic/Targets/SPIR.h:59 +// translation). This mapping is enabled when the language mode is HIP. +1, // cuda_device +// cuda_constant pointer can be casted to default/"flat" pointer, but in keryell

[PATCH] D108621: [HIPSPV] Add CUDA->SPIR-V address space mapping

keryell added inline comments. Comment at: clang/lib/Basic/Targets/SPIR.h:59 +// translation). This mapping is enabled when the language mode is HIP. +1, // cuda_device +// cuda_constant pointer can be casted to default/"flat" pointer, but in

[PATCH] D108370: [clang-tidy] Fix wrong FixIt about union in cppcoreguidelines-pro-type-member-init

Sockke added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/cppcoreguidelines-pro-type-member-init.cpp:534 + +union U2 { + U2() {} aaron.ballman wrote: > A related interesting test would be: > ``` > union U3 { > U3() {} > > struct

[PATCH] D109544: [OpenMP] Add flag for setting debug in the offloading device

mgorny added a comment. I'm sorry, it seems to have been caused by ccache. After clearing the cache, I can't reproduce anymore. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109544/new/ https://reviews.llvm.org/D109544

[PATCH] D106262: [clang][analyzer] Use generic note tag in alpha.unix.Stream .

Szelethus added a comment. I like everything I see here so far! As soon as those debug functions are in, the patch should land! Comment at: clang/lib/StaticAnalyzer/Checkers/StreamChecker.cpp:236-240 + /// At any stream operation that can cause (multiple type of) bugs, we

[PATCH] D105191: [Clang][OpenMP] Add support for Static Device Libraries

saiislam added a comment. In D105191#2996201 , @ye-luo wrote: > @saiislam do my test cases work on your side? I tried this patch and still > got linking failure. Hey @ye-luo, I am able to successfully compile and run all (36) tests in

[PATCH] D109659: Fix scan-build-py executable lookup path

This revision was automatically updated to reflect the committed changes. Closed by commit rGc84755a046bb: Fix scan-build-py executable lookup path (authored by serge-sans-paille). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109659/new/

[clang] c84755a - Fix scan-build-py executable lookup path

Author: serge-sans-paille Date: 2021-09-13T11:01:47+02:00 New Revision: c84755a046bbdcd0564693e30b2508034b06002b URL: https://github.com/llvm/llvm-project/commit/c84755a046bbdcd0564693e30b2508034b06002b DIFF:

[clang] bdce8d4 - Revert "[X86] Adjust Keylocker handle mem size"

Author: Xiang1 Zhang Date: 2021-09-13T18:00:46+08:00 New Revision: bdce8d40c6da56f1c95a8d7bfeac12b1ffce79cf URL: https://github.com/llvm/llvm-project/commit/bdce8d40c6da56f1c95a8d7bfeac12b1ffce79cf DIFF: https://github.com/llvm/llvm-project/commit/bdce8d40c6da56f1c95a8d7bfeac12b1ffce79cf.diff

[clang] c81d6ab - [X86] Adjust Keylocker handle mem size

Author: Xiang1 Zhang Date: 2021-09-13T18:03:27+08:00 New Revision: c81d6ab8758224ab950716d9533df79c5b5fb706 URL: https://github.com/llvm/llvm-project/commit/c81d6ab8758224ab950716d9533df79c5b5fb706 DIFF: https://github.com/llvm/llvm-project/commit/c81d6ab8758224ab950716d9533df79c5b5fb706.diff

[PATCH] D109488: [X86] Adjust Keylocker store register num for encodekey128/256

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc81d6ab87582: [X86] Adjust Keylocker handle mem size (authored by xiangzhangllvm). Herald added a project: clang. Herald added a subscriber:

[clang] d86a947 - [Sema] Add test for __builtin_fminf errors.

Author: Florian Hahn Date: 2021-09-13T11:12:06+01:00 New Revision: d86a947bb91cc67452057b4bc25c0c7734b5139c URL: https://github.com/llvm/llvm-project/commit/d86a947bb91cc67452057b4bc25c0c7734b5139c DIFF: https://github.com/llvm/llvm-project/commit/d86a947bb91cc67452057b4bc25c0c7734b5139c.diff

[PATCH] D109544: [OpenMP] Add flag for setting debug in the offloading device

mgorny added a comment. This change causes `clang-ast-dump` to segfault for me while building clang: [399/399] ASTNodeAPI.json FAILED: tools/clang/lib/Tooling/ASTNodeAPI.json /home/mgorny/git/llvm-project/build/tools/clang/lib/Tooling/ASTNodeAPI.json cd

[clang] 3731de6 - [X86] Adjust Keylocker handle mem size

Author: Xiang1 Zhang Date: 2021-09-13T17:59:33+08:00 New Revision: 3731de6b7f2d42d40151f9574636bc4d5ccfa5e3 URL: https://github.com/llvm/llvm-project/commit/3731de6b7f2d42d40151f9574636bc4d5ccfa5e3 DIFF: https://github.com/llvm/llvm-project/commit/3731de6b7f2d42d40151f9574636bc4d5ccfa5e3.diff

[PATCH] D109609: [C++4OpenCL] Add support for multiple address spaced destructors

keryell added a comment. While it might be possible to extend arbitrarily C++, I have the feeling that having just 1 destructor and have a different code path-code according the address space would not be enough. It is possible to write: ~MyDestructor() { if constexpr

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

curdeius added a comment. In D69764#2996365 , @steveire wrote: > FYI - there's nothing "anti-inclusive" about East/West. Thank you for your comment. Personally I don't think that the question of not using East/West is whether these terms are inclusive

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

foad added a comment. What is a "keep constructor"? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109483/new/ https://reviews.llvm.org/D109483 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D106343: [OpenCL] Support cl_ext_float_atomics

svenvh added a comment. Apologies for the delayed response. In D106343#2967089 , @haonanya wrote: > Hi, svenvh. > Should we use cl_khr_int64_base_atomics and cl_khr_int64_extended_atomics to > guard the functions using atomic_double type? > Thanks

[clang] ec6c847 - [clang] Check unsupported types in expressions

Author: Andrew Savonichev Date: 2021-09-13T14:59:37+03:00 New Revision: ec6c847179fd019acae4d97a18f9e7d3961a6fdf URL: https://github.com/llvm/llvm-project/commit/ec6c847179fd019acae4d97a18f9e7d3961a6fdf DIFF:

[PATCH] D109315: [clang] Check unsupported types in expressions

This revision was automatically updated to reflect the committed changes. Closed by commit rGec6c847179fd: [clang] Check unsupported types in expressions (authored by asavonic). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109315/new/

[clang] 9d359f6 - [analyzer] MallocChecker: Add notes from NoOwnershipChangeVisitor only when a function "intents", but doesn't change ownership, enable by default

Author: Kristóf Umann Date: 2021-09-13T15:01:20+02:00 New Revision: 9d359f6c738632c6973e9f5328b10bf39b3df55a URL: https://github.com/llvm/llvm-project/commit/9d359f6c738632c6973e9f5328b10bf39b3df55a DIFF: https://github.com/llvm/llvm-project/commit/9d359f6c738632c6973e9f5328b10bf39b3df55a.diff

[PATCH] D97699: [analyzer] Add InvalidPtrChecker

zukatsinadze marked 4 inline comments as done. zukatsinadze added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/cert/InvalidPtrChecker.cpp:163 +// memory region returned by previous call of this function +REGISTER_MAP_WITH_PROGRAMSTATE(PreviousCallResultMap,

[PATCH] D109544: [OpenMP] Add flag for setting debug in the offloading device

jdoerfert added a comment. In D109544#2996913 , @mgorny wrote: > I'm sorry, it seems to have been caused by ccache. After clearing the cache, > I can't reproduce anymore. The problem is that clang-ast-dump has incomplete dependences and sometimes

[clang] 8008009 - [OpenCL] Initialize temporaries in the private address space

Author: Ole Strohm Date: 2021-09-13T12:56:04+01:00 New Revision: 8008009fd25bf51c2c85c612bfefec64e975bbe4 URL: https://github.com/llvm/llvm-project/commit/8008009fd25bf51c2c85c612bfefec64e975bbe4 DIFF: https://github.com/llvm/llvm-project/commit/8008009fd25bf51c2c85c612bfefec64e975bbe4.diff

[PATCH] D107553: [C++4OpenCL] Initialize temporaries in the private address space

This revision was automatically updated to reflect the committed changes. Closed by commit rG8008009fd25b: [OpenCL] Initialize temporaries in the private address space (authored by olestrohm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

This revision was automatically updated to reflect the committed changes. Closed by commit rG648feabc65d8: [clang] Make the driver not diagnose errors on nonexistent linker inputs (authored by thakis). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM

[clang] 648feab - [clang] Make the driver not diagnose errors on nonexistent linker inputs

Author: Nico Weber Date: 2021-09-13T08:57:38-04:00 New Revision: 648feabc65d8ec20e5d39ac88e019d310955a6e6 URL: https://github.com/llvm/llvm-project/commit/648feabc65d8ec20e5d39ac88e019d310955a6e6 DIFF: https://github.com/llvm/llvm-project/commit/648feabc65d8ec20e5d39ac88e019d310955a6e6.diff

[PATCH] D108912: [release][analyzer] Add 13.0.0 release notes

Szelethus added a comment. Gonna land this in a day or two, regardless of whether its accepted! Please take a look if you have anything to object to! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108912/new/ https://reviews.llvm.org/D108912

[PATCH] D108893: clang-tidy: introduce readability-containter-data-pointer check

aaron.ballman added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/ContainerDataPointerCheck.cpp:56 +hasType(Container), +hasType(pointsTo(Container)), +

[PATCH] D109688: [AMDGPU] ASan Patches for OpenMP

ampandey-AMD created this revision. Herald added subscribers: kerbowa, guansong, hiraditya, t-tye, tpr, dstuttard, yaxunl, mgorny, nhaehnle, jvesely, kzhuravl. ampandey-AMD requested review of this revision. Herald added subscribers: llvm-commits, openmp-commits, cfe-commits, sstefan1, wdng.

[PATCH] D109611: Fix CLANG_ENABLE_STATIC_ANALYZER=OFF building all analyzer source

thakis added a comment. Can you just set `CLANG_TIDY_ENABLE_STATIC_ANALYZER=OFF` too if you care about this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109611/new/ https://reviews.llvm.org/D109611

[PATCH] D109611: Fix CLANG_ENABLE_STATIC_ANALYZER=OFF building all analyzer source

arichardson added a comment. In D109611#2997236 , @thakis wrote: > Can you just set `CLANG_TIDY_ENABLE_STATIC_ANALYZER=OFF` too if you care > about this? That doesn't have any effect on the libraries linked into clang-shlib. I am not building

[PATCH] D109437: [PowerPC] FP compare and test XL compat builtins.

quinnp updated this revision to Diff 372243. quinnp added a comment. Adding SemaChecking for the first argument of __builitn_ppc_test_data_class so the llvm_unreachable in CGBuiltin is actually unreachable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D108621: [HIPSPV] Add CUDA->SPIR-V address space mapping

linjamaki added inline comments. Comment at: clang/lib/Basic/Targets/SPIR.h:233 +if (Opts.HIP && Opts.CUDAIsDevice) + // Enable address space mapping from HIP to SPIR-V. + // See comment on the SPIRDefIsGenMap table. Anastasia wrote: > My guess is

[clang] 6377426 - Revert "[clang] Check unsupported types in expressions"

Author: Andrew Savonichev Date: 2021-09-13T15:34:21+03:00 New Revision: 6377426b4a326b52733065609a5d811afd2b8b1b URL: https://github.com/llvm/llvm-project/commit/6377426b4a326b52733065609a5d811afd2b8b1b DIFF:

[PATCH] D108753: [analyzer] MallocChecker: Add notes from NoOwnershipChangeVisitor only when a function "intents", but doesn't change ownership, enable by default

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9d359f6c7386: [analyzer] MallocChecker: Add notes from NoOwnershipChangeVisitor only when a… (authored by Szelethus). Repository: rG LLVM Github

[PATCH] D105191: [Clang][OpenMP] Add support for Static Device Libraries

ye-luo added a comment. @saiislam did you turn on offload? https://github.com/ye-luo/openmp-target/wiki/OpenMP-offload-compilers#llvm-clang On NVIDIA, it fails at CMake step. On AMD, make step stops because of unrelated issue. Please make the exact reproducer 1 working. Right now I got $

[PATCH] D97699: [analyzer] Add InvalidPtrChecker

zukatsinadze updated this revision to Diff 372237. zukatsinadze added a comment. Thanks for the review @martong I've fixed all the suggestions. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97699/new/ https://reviews.llvm.org/D97699 Files: clang/docs/analyzer/checkers.rst

[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

pengfei added a comment. In D109658#2996767 , @craig.topper wrote: > In D109658#2996714 , @pengfei wrote: > >> In D109658#2996412 , @craig.topper >> wrote: >> >>> Does

[clang] d353d1c - [OpenCL] Support cl_ext_float_atomics

Author: Sven van Haastregt Date: 2021-09-13T12:12:40+01:00 New Revision: d353d1c50112a1cb315eccdab18ce7bd1563cd06 URL: https://github.com/llvm/llvm-project/commit/d353d1c50112a1cb315eccdab18ce7bd1563cd06 DIFF:

[PATCH] D106343: [OpenCL] Support cl_ext_float_atomics

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd353d1c50112: [OpenCL] Support cl_ext_float_atomics (authored by svenvh). Herald added a subscriber: ldrumm. Changed prior to commit:

[PATCH] D108695: [analyzer][NFCI] Allow clients of NoStateChangeFuncVisitor to check entire function calls, rather than each ExplodedNode in it

This revision was automatically updated to reflect the committed changes. Closed by commit rG0213d7ec0c50: [analyzer][NFCI] Allow clients of NoStateChangeFuncVisitor to check entire… (authored by Szelethus). Changed prior to commit: https://reviews.llvm.org/D108695?vs=370549=372217#toc

[clang] 0213d7e - [analyzer][NFCI] Allow clients of NoStateChangeFuncVisitor to check entire function calls, rather than each ExplodedNode in it

Author: Kristóf Umann Date: 2021-09-13T13:50:01+02:00 New Revision: 0213d7ec0c501414d12020737fdc47e47e4392d9 URL: https://github.com/llvm/llvm-project/commit/0213d7ec0c501414d12020737fdc47e47e4392d9 DIFF: https://github.com/llvm/llvm-project/commit/0213d7ec0c501414d12020737fdc47e47e4392d9.diff

[PATCH] D107450: [clang-tidy] Fix wrong FIxIt in performance-move-const-arg

Sockke added a comment. kindly ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107450/new/ https://reviews.llvm.org/D107450 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D109686: Check supported architectures in sseXYZ/avxXYZ headers

serge-sans-paille created this revision. serge-sans-paille added reviewers: craig.topper, RKSimon. serge-sans-paille requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. It doesn't make sense to include those headers on the wrong architecture,

[PATCH] D109344: [AMDGPU][OpenMP] Use complex definitions from complex_cmath.h

pdhaliwal added a comment. Hey, Jon, sorry for late reply. I cannot reproduce this issue on nvptx so it seems to occur only on amdgcn. Will it be better if instead the name mangling issue is fixed? Or for the meantime, I could add #ifdef around as a temporary fix. Suggestions? Repository:

[PATCH] D106713: Thread safety analysis: Warn when demoting locks on back edges

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. Accepting the review -- if you don't hear back from @delesley in the next few days, I think it's fine to land. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109461: [clang][darwin] Add support for --emit-static-lib

yaxunl added inline comments. Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:748 + + // ar tool command "llvm-ar ". + ArgStringList CmdArgs; libtool ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97699: [analyzer] Add InvalidPtrChecker

martong accepted this revision. martong added a comment. This revision is now accepted and ready to land. Excellent! Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97699/new/ https://reviews.llvm.org/D97699 ___ cfe-commits mailing

[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

vaibhav.y created this revision. vaibhav.y added a reviewer: lattner. Herald added subscribers: dexonsmith, wenlei, mgorny. vaibhav.y edited the summary of this revision. vaibhav.y added a project: clang. vaibhav.y added subscribers: aaron.ballman, lebedev.ri. vaibhav.y added a comment. vaibhav.y

[PATCH] D109128: [VFS] Use original path when falling back to external FS

keith added a comment. In D109128#2997588 , @JDevlieghere wrote: > If I understand correctly, this patch does that, but swaps 2 and 3. What's > the motivation for trying the non-canonical path first? If we treat the > current working directory as a

[PATCH] D109461: [clang][darwin] Add support for --emit-static-lib

keith updated this revision to Diff 372287. keith added a comment. Revert unrelated formatting fixes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109461/new/ https://reviews.llvm.org/D109461 Files: clang/lib/Driver/ToolChain.cpp

[PATCH] D109344: [AMDGPU][OpenMP] Use complex definitions from complex_cmath.h

jdoerfert added a comment. In D109344#2997454 , @JonChesterfield wrote: > Let's change to the uglier #ifdef for now, and add it to the short list of > things that aren't quite right in declare variant Can you please file a bug with a reproducer, I'm

[PATCH] D109607: [X86] Refactor GetSSETypeAtOffset to fix pr51813

pengfei added inline comments. Comment at: clang/lib/CodeGen/TargetInfo.cpp:3454 - -return llvm::Type::getHalfTy(getVMContext()); - } LuoYuanke wrote: > Is this the major change? No, this logic is not changed. When `T1` is nullptr, it means `IRType` is

[PATCH] D106262: [clang][analyzer] Use generic note tag in alpha.unix.Stream .

balazske added a comment. Probably it is better to make a big comment at the start of the file that explains how the checker works, like in FuchsiaHandleChecker. This comes in a separate patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109642: -Wunused-but-set-parameter/-Wunused-but-set-variable Add to the release notes

mbenfield added a comment. I'd request small changes for grammar: `Wunused-but-set-parameter` and `-Wunused-but-set-variable` emit warnings when a parameter or a variable is set but not used. Otherwise LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D109608: [clang][ASTImporter] Generic attribute import handling (first step).

shafik added inline comments. Comment at: clang/lib/AST/ASTImporter.cpp:711 + +/*template +friend struct AttrArgImporter; Looks like left over unused code? Comment at: clang/lib/AST/ASTImporter.cpp:8448 + +class AttrImporter { + Error

[PATCH] D109557: Adds an AlignCloseBracket option

MyDeveloperDay added a comment. Please resubmit the patch with full context Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109557/new/ https://reviews.llvm.org/D109557 ___ cfe-commits mailing list

[PATCH] D109688: [AMDGPU] ASan Patches for OpenMP

jdoerfert requested changes to this revision. jdoerfert added a comment. This revision now requires changes to proceed. This doesn't seem to be based on LLVM upstream, among other issues. Blocking it for now w/o looking at the actual patch. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

RKSimon added a comment. > Regarding this case, we already have a different name with GCC, I think it > worthwhile to use a different order for the swapping optimization. > With a bit research on AVX512IFMA, I found: > > 1. The use of C, A, B order in GCC is not consistent on its AVX512IFMA >

[PATCH] D109128: [VFS] Use original path when falling back to external FS

JDevlieghere added a comment. Keith and I discussed this offline. My suggestion was to do the following: 1. Check the overlay for the canonicalized path 2. Check the fall-through for the canonicalized path 3. Check the fall-through for the original path If I understand correctly, this patch

[PATCH] D109608: [clang][ASTImporter] Generic attribute import handling (first step).

martong added a reviewer: teemperor. martong added a comment. Thanks, nice work! I like the direction, however, I'd like to ask comments from @teemperor . Comment at: clang/lib/AST/ASTImporter.cpp:656-671 +// Helper for chaining together multiple imports. If an error is

[PATCH] D109344: [AMDGPU][OpenMP] Use complex definitions from complex_cmath.h

JonChesterfield added a comment. In D109344#2997495 , @jdoerfert wrote: > Can you please file a bug with a reproducer, I'm not sure I follow your > problem. Should be a case of trying to compile complex test cases for ovo for amdgpu immediately

[PATCH] D69764: [clang-format] Add Left/Right Const fixer capability

aaron.ballman added a comment. In D69764#2996365 , @steveire wrote: > FYI - there's nothing "anti-inclusive" about East/West. While I'm certain that there can be confusion around why terms are or are not troublesome to some people and there needs to be

[PATCH] D104556: [InstrProfiling] Make CountersPtr in __profd_ relative

catenacyber added a comment. This commit has broken oss-fuzz workflow for rust projects. I do not know if the fix is https://reviews.llvm.org/rG66e2772e4285588ccc3bcdb5f392c8326debbd7d Scenario is export RUSTFLAGS="--cfg fuzzing -Cdebuginfo=1 -Cforce-frame-pointers -Zinstrument-coverage -C

[PATCH] D109483: [APInt] Normalize naming on keep constructors / predicate methods.

lattner added a comment. > What is a "keep constructor"? Good question, I'm not sure. I think I meant to say "key constructors". Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109483/new/ https://reviews.llvm.org/D109483

[PATCH] D109694: Driver: Add preferred gcc triples for Red Hat operating systems

MaskRay added a comment. Haven't looked closely at this particular commit, but I think hard coding the detected triples is almost always a bad idea. See https://reviews.llvm.org/D63497#2993662 for a longer comment. The proper way is (1) making sure CMake `LLVM_DEFAULT_TARGET_TRIPLE` is correct

[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

craig.topper added a comment. In D109658#2997395 , @pengfei wrote: > In D109658#2996767 , @craig.topper > wrote: > >> In D109658#2996714 , @pengfei >> wrote: >> >>> In

[PATCH] D109344: [AMDGPU][OpenMP] Use complex definitions from complex_cmath.h

JonChesterfield added a comment. Let's change to the uglier #ifdef for now, and add it to the short list of things that aren't quite right in declare variant Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109344/new/

[PATCH] D109607: [X86] Refactor GetSSETypeAtOffset to fix pr51813

pengfei updated this revision to Diff 372254. pengfei marked an inline comment as done. pengfei added a comment. Add more comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109607/new/ https://reviews.llvm.org/D109607 Files:

[PATCH] D109694: Driver: Add preferred gcc triples for Red Hat operating systems

tstellar created this revision. tstellar added reviewers: serge-sans-paille, MaskRay, mgorny. Herald added a subscriber: pengfei. tstellar requested review of this revision. Herald added a project: clang. The GCCInstallationDetector has a list of gcc triples to search for when trying to find a

[PATCH] D108556: [clangd] Don't highlight ObjC `id` and `instancetype`

dgoldman marked an inline comment as done. dgoldman added a comment. Friendly ping, PTAL Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108556/new/ https://reviews.llvm.org/D108556 ___ cfe-commits

[PATCH] D109531: [CSSPGO] Enable pseudo probe instrumentation in O0 mode.

hoy added inline comments. Comment at: llvm/lib/Passes/PassBuilder.cpp:1930 + // mixed with an O0 prelink and an O2 postlink. Loading a sample profile in + // the postlink will require pseudo probe instrumentation in the prelink. + if (PGOOpt &&

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

phosek added a comment. `Clang :: Driver/cl.c` started failing on our Windows bots after this change landed. Here

[PATCH] D109693: Driver: Remove major release version detection for RHEL

tstellar created this revision. tstellar added a reviewer: serge-sans-paille. tstellar requested review of this revision. Herald added a project: clang. We treat all release of RHEL the same in the driver, so we don't need to detect the RHEL major release version. Repository: rG LLVM Github

[PATCH] D97699: [analyzer] Add InvalidPtrChecker

steakhal added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/CMakeLists.txt:42 DynamicTypeChecker.cpp + cert/InvalidPtrChecker.cpp EnumCastOutOfRangeChecker.cpp Please, insert this in its sorted place. CHANGES SINCE LAST ACTION

[PATCH] D104556: [InstrProfiling] Make CountersPtr in __profd_ relative

MaskRay added a comment. In D104556#2997674 , @catenacyber wrote: > This commit has broken oss-fuzz workflow for rust projects. > I do not know if the fix is > https://reviews.llvm.org/rG66e2772e4285588ccc3bcdb5f392c8326debbd7d > > Scenario is > >

[PATCH] D109703: [DebugInfo] Fix scope for local static variables

krisb created this revision. krisb added reviewers: dblaikie, probinson. Herald added a subscriber: hiraditya. krisb requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. This fixes https://bugs.llvm.org/show_bug.cgi?id=44695

[PATCH] D109461: [clang][darwin] Add support for --emit-static-lib

keith updated this revision to Diff 372286. keith marked an inline comment as done. keith added a comment. Fix formatting and comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109461/new/ https://reviews.llvm.org/D109461 Files:

[PATCH] D108893: clang-tidy: introduce readability-containter-data-pointer check

compnerd updated this revision to Diff 372285. compnerd added a comment. Address review feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108893/new/ https://reviews.llvm.org/D108893 Files:

[PATCH] D108893: clang-tidy: introduce readability-containter-data-pointer check

compnerd marked 7 inline comments as done. compnerd added inline comments. Comment at: clang-tools-extra/clang-tidy/readability/ContainerDataPointerCheck.cpp:63 +hasType(Container), +

[PATCH] D108243: Put code that avoids heapifying local blocks behind a flag

waltl updated this revision to Diff 372383. waltl added a comment. Mark new option as NoDriverOption Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108243/new/ https://reviews.llvm.org/D108243 Files: clang/include/clang/Basic/CodeGenOptions.def

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

haonanya created this revision. haonanya added reviewers: svenvh, Anastasia. haonanya added a project: clang. Herald added subscribers: ldrumm, yaxunl. haonanya requested review of this revision. Herald added a subscriber: cfe-commits. Add atomic_half types and builtins operating on the types

[PATCH] D109658: [X86][FP16] Change the order of the operands in complex FMA intrinsics to allow swap between the mul operands.

pengfei updated this revision to Diff 372400. pengfei added a comment. Allow MI operands of complex FMA to be commutable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109658/new/ https://reviews.llvm.org/D109658 Files:

[PATCH] D109144: [SPIR-V] Add SPIR-V triple architecture and clang target info

linjamaki updated this revision to Diff 372406. linjamaki added a comment. Add -verify and expected-no-diagnostics to a SPIR-V test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109144/new/ https://reviews.llvm.org/D109144 Files:

[clang] 2bd8493 - Improve type printing of const arrays to normalize array-of-const and const-array

Author: David Blaikie Date: 2021-09-13T19:17:05-07:00 New Revision: 2bd84938470bf2e337801faafb8a67710f46429d URL: https://github.com/llvm/llvm-project/commit/2bd84938470bf2e337801faafb8a67710f46429d DIFF: https://github.com/llvm/llvm-project/commit/2bd84938470bf2e337801faafb8a67710f46429d.diff

[PATCH] D36850: [ThinLTO] Add norecurse function attribute propagation

modimo updated this revision to Diff 372398. modimo added a comment. Add mayThrow flag Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36850/new/ https://reviews.llvm.org/D36850 Files: clang/test/CodeGen/thinlto-distributed-cfi-devirt.ll

[PATCH] D104556: [InstrProfiling] Make CountersPtr in __profd_ relative

catenacyber added a comment. > .profraw are files with the raw profile format. The compiler-rt/lib/profile > runtime and llvm-profdata only support one version at any commit. Ok. > Mixing .profraw files produced by different compiler-rt/lib/profile runtimes > is unsupported. I do not think I

[PATCH] D104556: [InstrProfiling] Make CountersPtr in __profd_ relative

MaskRay added a comment. In D104556#2998159 , @catenacyber wrote: >> .profraw are files with the raw profile format. The compiler-rt/lib/profile >> runtime and llvm-profdata only support one version at any commit. > > Ok. > >> Mixing .profraw files

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

gandhi21299 marked an inline comment as done. gandhi21299 added inline comments. Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5069 // where aliases aren't supported. - if (!RawTriple.isOSDarwin() && !RawTriple.isNVPTX() && !RawTriple.isAMDGPU())

[PATCH] D109714: Reduce maximum width of _ExtInt from 2^24-1 to 2^23 to workaround SelectionDAG issue.

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM modulo a request for some comments in the code. It makes me sad that we have a max number of bits specified by LLVM that's kinda useless. "You can store this many bits into this integer value, but if you attempt to add 0

[PATCH] D109714: Reduce maximum width of _ExtInt from 2^24-1 to 2^23 to workaround SelectionDAG issue.

craig.topper updated this revision to Diff 372331. craig.topper added a comment. Add comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109714/new/ https://reviews.llvm.org/D109714 Files: clang/docs/ReleaseNotes.rst

[PATCH] D109703: [DebugInfo] Fix scope for local static variables

krisb added a comment. In D109703#2998155 , @ellis wrote: > I've added this to the added test case. > > !18 = !DIImportedEntity(tag: DW_TAG_imported_declaration, scope: !4, > entity: !19, line: 122) > !19 = !DIGlobalVariable(name:

[PATCH] D109686: Check supported architectures in sseXYZ/avxXYZ headers

craig.topper accepted this revision. craig.topper 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/D109686/new/ https://reviews.llvm.org/D109686

[PATCH] D108893: clang-tidy: introduce readability-containter-data-pointer check

compnerd marked an inline comment as done. compnerd added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/readability-container-data-pointer.cpp:104 + // CHECK-MESSAGES-NOT: :[[@LINE-1]]:5: warning: 'data' should be used for accessing the data pointer

[clang] 0867c01 - [ClangScanDeps][test] Add -fmodules-cache-path=DIR/cache to make diagnostics.c hermetic

Author: Fangrui Song Date: 2021-09-13T13:17:55-07:00 New Revision: 0867c017e55e3a3d457f26bb778a2cd80d86e1f4 URL: https://github.com/llvm/llvm-project/commit/0867c017e55e3a3d457f26bb778a2cd80d86e1f4 DIFF: https://github.com/llvm/llvm-project/commit/0867c017e55e3a3d457f26bb778a2cd80d86e1f4.diff

  1   2   >