[PATCH] D94376: [MemCpyOpt] Enable MemorySSA by default

2021-02-22 Thread Mikael Holmén via Phabricator via cfe-commits
uabelho added a comment. In D94376#2579242 , @nikic wrote: > @uabelho Thanks for the report! This issue should be resolved by > https://github.com/llvm/llvm-project/commit/4125afc35723b490556f7a38f7835f0914f70292. Yep, thanks! Repository: rG LLVM Git

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

2021-02-22 Thread Florian Schmaus via Phabricator via cfe-commits
Flow added a comment. In D90972#2580020 , @njames93 wrote: > In D90972#2446448 , @Flow wrote: > >> In D90972#2394516 , @JonasToth >> wrote: >> >>> LGTM! thanks for fixing. >

[PATCH] D96110: [X86] Pass to transform tdpbf16ps intrinsics to scalar operation.

2021-02-22 Thread Bing Yu via Phabricator via cfe-commits
yubing updated this revision to Diff 325683. yubing added a comment. Rebase and add a testcase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96110/new/ https://reviews.llvm.org/D96110 Files: clang/include/clang/Basic/BuiltinsX86_64.def clang/

[PATCH] D65616: Ignore -fsemantic-interposition/-fno-semantic-interposition flag for gcc compatibility

2021-02-22 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay commandeered this revision. MaskRay edited reviewers, added: Romain-Geissler-1A; removed: MaskRay. MaskRay added a comment. This revision is now accepted and ready to land. Herald added subscribers: jansvoboda11, dang. Implemented by 2820a2ca3a0e69c3f301845420e00672251b

[PATCH] D91844: [llvm][clang] Add checks for the smart pointers with the possibility to be null

2021-02-22 Thread Ella Ma via Phabricator via cfe-commits
OikawaKirie abandoned this revision. OikawaKirie added a comment. Split to - D97251 (clang/utils/TableGen/ClangAttrEmitter.cpp) - D97185 (llvm/lib/DWARFLinker/DWARFLinker.cpp) - D97254 (llvm/l

[PATCH] D96110: [X86] Pass to transform tdpbf16ps intrinsics to scalar operation.

2021-02-22 Thread Bing Yu via Phabricator via cfe-commits
yubing updated this revision to Diff 325685. yubing added a comment. Fix incorrect naming for dpbf16's bb Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96110/new/ https://reviews.llvm.org/D96110 Files: clang/include/clang/Basic/BuiltinsX86_64.de

[PATCH] D97259: [X86] Support amx-int8 intrinsic.

2021-02-22 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 created this revision. Herald added subscribers: pengfei, hiraditya. LiuChen3 requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Adding support for intrinsics of TDPBSUD/TDPBUSD/TDPBUUD. Repository: rG LLVM Githu

[PATCH] D97259: [X86] Support amx-int8 intrinsic.

2021-02-22 Thread LiuChen via Phabricator via cfe-commits
LiuChen3 added inline comments. Comment at: clang/include/clang/Basic/BuiltinsX86_64.def:106 TARGET_BUILTIN(__builtin_ia32_tdpbssd_internal, "V256iUsUsUsV256iV256iV256i", "n", "amx-int8") + +TARGET_BUILTIN(__builtin_ia32_tdpbsud_internal, "V256iUsUsUsV256iV256iV256i", "n", "am

[PATCH] D96110: [X86] Pass to transform tdpbf16ps intrinsics to scalar operation.

2021-02-22 Thread Bing Yu via Phabricator via cfe-commits
yubing updated this revision to Diff 325688. yubing added a comment. Modify some comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96110/new/ https://reviews.llvm.org/D96110 Files: clang/include/clang/Basic/BuiltinsX86_64.def clang/lib/He

[PATCH] D96320: [ThinLTO, NewPM] Run OptimizerLastEPCallbacks from buildThinLTOPreLinkDefaultPipeline

2021-02-22 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added a comment. Sorry, just got back from vacation, I'll try to take a look tomorrow Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96320/new/ https://reviews.llvm.org/D96320 ___ cfe-commits mai

[PATCH] D97182: [clang] fix cmake wrong variable name from RTLIBS to RTLIB

2021-02-22 Thread Philip Lassen via Phabricator via cfe-commits
philass updated this revision to Diff 325690. philass added a comment. Rerun failed build that may be fixed in https://reviews.llvm.org/rG4827492d9fcfe4ac5df4f319e935e33bec946195 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97182/new/ https://reviews.llvm.org/D97182 Files: clang/CM

<    1   2   3