[PATCH] D154869: [Flang] [FlangRT] Introduce FlangRT project as solution to Flang's runtime LLVM integration

2023-10-01 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. In D154869#4652224 , @sylvestre.ledru wrote: > With this patch, it started to fail with: > It would be nice to have a better error message suggesting what to do > > -- Performing Test HAVE_DECL_STRERROR_S > -- Performing Test

[PATCH] D154869: [Flang] [FlangRT] Introduce FlangRT project as solution to Flang's runtime LLVM integration

2023-08-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang-rt/unittests/FortranEvaluate/testing.h:1 +#ifndef FORTRAN_EVALUATE_TESTING_H_ +#define FORTRAN_EVALUATE_TESTING_H_ klausler wrote: > Why was this file (and testing.cpp) added instead of being moved, like other >

[PATCH] D154869: [Flang] [FlangRT] Introduce FlangRT project as solution to Flang's runtime LLVM integration

2023-08-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. Ping I will not be able to help out with flang (for a while) after the end of August so the sooner I can get this reviewed the smoother the landing process will be for myself and the community. So I would appreciate if I could get reviewers looking at this as soon as

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-08-15 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/lib/Decimal/CMakeLists.txt:52 -add_flang_library(FortranDecimal INSTALL_WITH_TOOLCHAIN - binary-to-decimal.cpp - decimal-to-binary.cpp -) +add_compile_options(-fPIC) + efriedma wrote: > This

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-08-15 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. In order to keep the scope of this patch manageable, and because further discussion is warranted first, no source files have been moved from flang into flang-rt. Although I would imagine that this may be something we would want to do in the future in order to continue

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-08-15 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. Some of the testcases have been reformatted by clang-format when I updated the patch with arcanist. If the community is against introducing extra changes like these I would be happy to re-update my patch without clang-formatting these files. Repository: rG LLVM

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-08-15 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/runtime/CMakeLists.txt:251 - INSTALL_WITH_TOOLCHAIN -) +if (DEFINED LLVM_ENABLE_RUNTIMES AND "flang-rt" IN_LIST LLVM_ENABLE_RUNTIMES) + add_flang_library(FortranRuntime STATIC efriedma wrote: > klausler wrote:

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-26 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/runtime/CMakeLists.txt:251 - INSTALL_WITH_TOOLCHAIN -) +if (DEFINED LLVM_ENABLE_RUNTIMES AND "flang-rt" IN_LIST LLVM_ENABLE_RUNTIMES) + add_flang_library(FortranRuntime STATIC efriedma wrote: > pscoro wrote: >

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-19 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/runtime/CMakeLists.txt:251 - INSTALL_WITH_TOOLCHAIN -) +if (DEFINED LLVM_ENABLE_RUNTIMES AND "flang-rt" IN_LIST LLVM_ENABLE_RUNTIMES) + add_flang_library(FortranRuntime STATIC efriedma wrote: > pscoro wrote: >

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/runtime/CMakeLists.txt:251 - INSTALL_WITH_TOOLCHAIN -) +if (DEFINED LLVM_ENABLE_RUNTIMES AND "flang-rt" IN_LIST LLVM_ENABLE_RUNTIMES) + add_flang_library(FortranRuntime STATIC efriedma wrote: > This "if"

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro marked 3 inline comments as done. pscoro added inline comments. Comment at: flang-rt/CMakeLists.txt:17-23 +# Check if flang-rt is built as a standalone project. +if (CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR OR FLANG_RT_STANDALONE_BUILD) + project(FlangRT C

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 541544. pscoro marked an inline comment as done. pscoro added a comment. Removed support for "fully standalone" Flang-rt builds (ie CMake builds targeting llvm-project/flang-rt). Flang-rt is now buildable via the `llvm` target or the `runtimes` target. See

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro marked 6 inline comments as done. pscoro added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:934-936 + // Default to the /../lib and + // /../runtimes/runtimes-bins/lib directories. This works fine + // on the platforms that we have tested so

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 541537. pscoro marked an inline comment as done. pscoro added a comment. Updated documentation to address all nits Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-17 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro marked an inline comment as done. pscoro added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:934-936 + // Default to the /../lib and + // /../runtimes/runtimes-bins/lib directories. This works fine + // on the platforms that we have tested so

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-17 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 541040. pscoro added a comment. Added documentation for Flang-rt. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-14 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 540413. pscoro added a comment. Decided to remove all the commented code related to the previous approach (wrapper library). The code that exists currently is for the object library approach and is functional. Currently, on phabricator there is a failing

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-13 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang-rt/CMakeLists.txt:17-23 +# Check if flang-rt is built as a standalone project. +if (CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR OR FLANG_RT_STANDALONE_BUILD) + project(FlangRT C CXX) + set(CMAKE_INCLUDE_CURRENT_DIR ON) +

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-13 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 540110. pscoro added a comment. Fix build failures Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-13 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 540026. pscoro added a comment. Fix driver usage of flang-rt Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files: clang/lib/Driver/ToolChains/CommonArgs.cpp

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-12 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 539545. pscoro added a comment. Small changes, flang-rt seems to work with the flang-new driver now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files:

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-11 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 539113. pscoro added a comment. Some small changes, still non functional Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files:

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-10 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 538749. pscoro added a comment. Comment out some code that irrelevant to current object library approach Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154869/new/ https://reviews.llvm.org/D154869 Files:

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-10 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: flang/cmake/modules/AddFlang.cmake:68 llvm_add_library(${name} ${LIBTYPE} ${ARG_UNPARSED_ARGUMENTS} ${srcs}) + set_target_properties(${name} PROPERTIES POSITION_INDEPENDENT_CODE ON) This is something I am still

[PATCH] D154869: [Flang] [FlangRT] Implement FlangRT library as solution to Flang's runtime LLVM integration

2023-07-10 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro created this revision. Herald added subscribers: sunshaoce, mstorsjo. Herald added a reviewer: sscalpone. Herald added a reviewer: awarzynski. Herald added projects: Flang, All. pscoro requested review of this revision. Herald added subscribers: llvm-commits, libcxx-commits, cfe-commits,

[PATCH] D140628: [flang] Add driver install directory to AIX toolchain program paths list

2023-01-24 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140628/new/ https://reviews.llvm.org/D140628 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D140628: [flang] Add driver install directory to AIX toolchain program paths list

2023-01-18 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. I have added a couple more reviewers based on revision history of the file being modified. If anybody is aware of anybody else who should be aware of this revision that would be appreciated Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D140628: [flang] Add driver install directory to AIX toolchain program paths list

2023-01-16 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. Herald added a subscriber: sunshaoce. Ping. This is a needed fix for AIX, the code is trivial and used similarly on other targets. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140628/new/ https://reviews.llvm.org/D140628

[PATCH] D140628: [flang] Add driver install directory to AIX toolchain program paths list

2022-12-23 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro created this revision. Herald added a project: All. pscoro requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D140628 Files: clang/lib/Driver/ToolChains/AIX.cpp

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-17 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 453272. pscoro added a comment. Reverted chain revision again, final Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files:

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-16 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 453105. pscoro added a comment. Re added chain fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-16 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 452990. pscoro added a comment. reverted previous fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-15 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 452740. pscoro added a comment. chain input fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-09 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 451133. pscoro added a comment. Small revisions Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-03 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 449683. pscoro added a comment. removed comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-02 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro marked 8 inline comments as done. pscoro added inline comments. Comment at: llvm/test/CodeGen/PowerPC/kill-canary-intrinsic.ll:11 + +; RUN: llc -verify-machineinstrs -mtriple=powerpc-unknown-aix -ppc-vsr-nums-as-vr \ +; RUN: -mcpu=pwr8 --ppc-asm-full-reg-names < %s |

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-08-02 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 449299. pscoro marked 3 inline comments as done. pscoro added a comment. Fixed test cases and AIX implementation Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-23 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 447102. pscoro added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-19 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 445853. pscoro added a comment. formatted comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: llvm/lib/Target/PowerPC/PPCISelLowering.cpp Index:

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-11 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 443630. pscoro added a comment. fixed lit test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: llvm/include/llvm/Debuginfod/Debuginfod.h:34 #include +#include #include Im really unsure how this change got in here, this is not added by me and on a freshly created branch and pull Any ideas? I'll look into

[PATCH] D128652: [PowerPC] Finished kill_canary implementation and debugging

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442682. pscoro added a comment. patch fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128652/new/ https://reviews.llvm.org/D128652 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442678. pscoro added a comment. patch fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442677. pscoro added a comment. patch fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125916/new/ https://reviews.llvm.org/D125916 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442643. pscoro added a comment. patch application Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442640. pscoro added a comment. trying to get patch to apply Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442630. pscoro added a comment. patch appication troubleshooting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442627. pscoro added a comment. rebased Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-06 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442615. pscoro added a comment. trying to localize calng-format again Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files:

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-05 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442368. pscoro added a comment. removed commented code Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-05 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 442320. pscoro added a comment. small revisions Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-01 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 441793. pscoro added a comment. Combined aix and linux IR lit tests into a single file Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129016/new/ https://reviews.llvm.org/D129016 Files:

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-01 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added inline comments. Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:11150 + +const uint64_t XORWord = 0x; // XORing with 0b111...111 will never + // result in the original word To address a

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-01 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. Sorry about making a separate review for this again, I encountered some troubles with git and decided that moving my code to a fresh branch would be quicker than figuring out how to fix the problem, the previous review will be abandoned. Repository: rG LLVM Github

[PATCH] D129016: [PowerPC] implemented @llvm.ppc.kill.canary to corrupt stack guard

2022-07-01 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro created this revision. Herald added subscribers: shchenz, kbarton, hiraditya, nemanjai. Herald added a project: All. pscoro requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo

[PATCH] D128652: [PowerPC] Finished kill_canary implementation and debugging

2022-06-30 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 441374. pscoro added a comment. fixed XOR usage, added linux tests and other small fixes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128652/new/ https://reviews.llvm.org/D128652 Files:

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-06-27 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro abandoned this revision. pscoro added inline comments. Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:11136 +MachineFrameInfo = MF.getFrameInfo(); +int SPI = MFI.getStackProtectorIndex(); // should return -1 + shchenz wrote: > Why

[PATCH] D128652: [PowerPC] Finished kill_canary implementation and debugging Merge branch 'Paul_3105final' of github.ibm.com:compiler/llvm-project into Paul_3105final

2022-06-27 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro created this revision. Herald added subscribers: shchenz, kbarton, hiraditya, nemanjai. Herald added a project: All. pscoro requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. Repository: rG LLVM Github Monorepo

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-06-22 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro added a comment. @shchenz I have added the test cases. This intrinsic is defined in order to test the stackprotect functionality on xlf. Use the -qdebug=smashstack flag to use it, make sure that -qstackprotect=all is also set Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-06-22 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 438988. pscoro added a comment. [PowerPC] Removed commented change Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125916/new/ https://reviews.llvm.org/D125916 Files: clang/include/clang/Basic/BuiltinsPPC.def

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-06-22 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 438987. pscoro added a comment. [PowerPC] Removed accidental patch files Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125916/new/ https://reviews.llvm.org/D125916 Files:

[PATCH] D125916: [PowerPC] Defined and lowered llvm.ppc.kill.canary intrinsic

2022-06-22 Thread Paul Scoropan via Phabricator via cfe-commits
pscoro updated this revision to Diff 438985. pscoro added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. [PowerPC] implemented kill_canary intrinsic and tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION