[PATCH] D106277: [SVE] Remove the interface for in favour of the IR attributes

2021-07-19 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm created this revision. Herald added subscribers: frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, niosHD, sabuasal, simoncook, johnrusso, rbar, asb, hiraditya, tschu

[PATCH] D106277: [SVE] Remove the interface for getMaxVScale in favour of the IR attributes

2021-07-22 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 360879. DylanFleming-arm added a comment. Removed changes to RiscV code Added check that target isAArch64 before adding default value vscale_range attribute Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D106277: [SVE] Remove the interface for getMaxVScale in favour of the IR attributes

2021-07-26 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 361675. DylanFleming-arm added a comment. Added getVScaleRange interface to TargetInfo and removed related AArch64 specific code from CodeGenFunction.cpp Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1

[PATCH] D106277: [SVE] Remove the interface for getMaxVScale in favour of the IR attributes

2021-07-27 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 362020. DylanFleming-arm added a comment. Rebased onto main, updated newly added AArch64 getMaxVScale usages to use IR attribute instead Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106277/new/ https

[PATCH] D129864: [Flang] Generate documentation for compiler flags

2022-07-15 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm created this revision. Herald added subscribers: arphaman, mgorny. Herald added a reviewer: sscalpone. Herald added projects: Flang, All. DylanFleming-arm requested review of this revision. Herald added subscribers: cfe-commits, jdoerfert. Herald added a project: clang. This patch

[PATCH] D129864: [Flang] Generate documentation for compiler flags

2022-07-19 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 445872. DylanFleming-arm added a comment. Edited summary to be more clear. I've also changed the format of the if(!included) as you suggested (and did the same for the call to isGroupIncluded) I also added the assert you asked for, and then since t

[PATCH] D129864: [Flang] Generate documentation for compiler flags

2022-07-21 Thread Dylan Fleming via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG396e944d82f3: [Flang] Generate documentation for compiler flags (authored by DylanFleming-arm). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129864/new/ ht

[PATCH] D129864: [Flang] Generate documentation for compiler flags

2022-07-22 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 446840. DylanFleming-arm added a comment. After pushing to main, this patch cause a buildbot failure as CLANG_TABLEGEN_EXE could not be found. I've updated flang/docs/CMakeLists.txt to set the parameter before making a call to clang_tablegen Repo

[PATCH] D129864: [Flang] Generate documentation for compiler flags

2022-07-22 Thread Dylan Fleming via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG846439dd97d4: [Flang] Generate documentation for compiler flags (authored by DylanFleming-arm). Changed prior to commit: https://reviews.llvm.org/D129864?vs=446840&id=446889#toc Repository: rG LLVM G

[PATCH] D106277: [SVE] Remove the interface for getMaxVScale in favour of the IR attributes

2021-08-12 Thread Dylan Fleming via Phabricator via cfe-commits
DylanFleming-arm updated this revision to Diff 366001. DylanFleming-arm added a comment. Added checks for MaxVScale > 0 Changed getMaxNumElemenets() to take Function* instead of Instruction* Fixed clang-tidy warning I haven't included a test for vscale_range(2, 0) here, as one was added in the m

[PATCH] D106277: [SVE] Remove the interface for getMaxVScale in favour of the IR attributes

2021-08-17 Thread Dylan Fleming via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGef198cd99e6b: [SVE] Remove usage of getMaxVScale for AArch64, in favour of IR Attribute (authored by DylanFleming-arm). Changed prior to commit: h