Page MenuHomePhabricator

reames (Philip Reames)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 23 2013, 6:32 PM (453 w, 6 h)

Recent Activity

Yesterday

reames updated the summary of D128840: [lit] Pass through QEMU's configuration environmental variables.
Wed, Jun 29, 11:46 AM · Restricted Project, Restricted Project
reames requested review of D128840: [lit] Pass through QEMU's configuration environmental variables.
Wed, Jun 29, 11:10 AM · Restricted Project, Restricted Project
reames added a comment to D128006: [RISCV] Avoid changing etype for splat of 0 or -1.

Fraser reported another problem with this patch to me privately. In addition to the wrong number of bits being written Craig found, we could also construct illegal instruction encodings. This could happen because we changed the VLMUL of the splat, but did not change the LMUL on the pseudo itself. As a result, the register allocator would assign e.g. an LMUL1 register, when the VSETVLI was now e.g. LMUL2. Given only have of registers are legal operands at LMUL2, this has a high chance of producing an illegal instruction encoding.

Wed, Jun 29, 10:55 AM · Restricted Project, Restricted Project
reames added a reverting change for D128006: [RISCV] Avoid changing etype for splat of 0 or -1: rGdd48d3ad0e5f: Revert "[RISCV] Avoid changing etype for splat of 0 or -1".
Wed, Jun 29, 10:28 AM · Restricted Project, Restricted Project
reames added a reverting change for rG755c84c62cda: [RISCV] Avoid changing etype for splat of 0 or -1: rGdd48d3ad0e5f: Revert "[RISCV] Avoid changing etype for splat of 0 or -1".
Wed, Jun 29, 10:28 AM · Restricted Project, Restricted Project
reames committed rGdd48d3ad0e5f: Revert "[RISCV] Avoid changing etype for splat of 0 or -1" (authored by reames).
Revert "[RISCV] Avoid changing etype for splat of 0 or -1"
Wed, Jun 29, 10:27 AM · Restricted Project, Restricted Project
reames added inline comments to D128006: [RISCV] Avoid changing etype for splat of 0 or -1.
Wed, Jun 29, 10:17 AM · Restricted Project, Restricted Project

Tue, Jun 28

reames committed rGda60558d8aa9: [RISCV] Rename getMin/MaxVLen to getArchMin/MaxVlen and make protected [nfc] (authored by reames).
[RISCV] Rename getMin/MaxVLen to getArchMin/MaxVlen and make protected [nfc]
Tue, Jun 28, 3:55 PM · Restricted Project, Restricted Project
reames committed rG860c62f53cb2: [RISCV] Refine known bits for READ_VLENB (authored by reames).
[RISCV] Refine known bits for READ_VLENB
Tue, Jun 28, 3:42 PM · Restricted Project, Restricted Project
reames closed D128758: [RISCV] Refine known bits for READ_VLENB.
Tue, Jun 28, 3:42 PM · Restricted Project, Restricted Project
reames updated the diff for D128758: [RISCV] Refine known bits for READ_VLENB.

Update per review comments.

Tue, Jun 28, 2:09 PM · Restricted Project, Restricted Project
reames added inline comments to D128758: [RISCV] Refine known bits for READ_VLENB.
Tue, Jun 28, 2:00 PM · Restricted Project, Restricted Project
reames committed rGf239cddbac3c: [RISCV] Pin two tests to fixed length vectorization to preserve test intent (authored by reames).
[RISCV] Pin two tests to fixed length vectorization to preserve test intent
Tue, Jun 28, 1:53 PM · Restricted Project, Restricted Project
reames accepted D128741: [RISCV] Fold (add X, [-4096, -2049]) or (add X, [2048,4096]) into load/store address during isel..

LGTM

Tue, Jun 28, 1:43 PM · Restricted Project, Restricted Project
reames added inline comments to D128758: [RISCV] Refine known bits for READ_VLENB.
Tue, Jun 28, 1:38 PM · Restricted Project, Restricted Project
reames requested changes to D128717: [RISCV] Change VECTOR_SPLICE mask operation from expand to promote.

Please land the new tests, and then rebase so that code changes are visible. (Unless this is fixing a crash, and then please clarify the commit message.)

Tue, Jun 28, 1:29 PM · Restricted Project, Restricted Project
reames updated the diff for D128758: [RISCV] Refine known bits for READ_VLENB.

Remove stray include change

Tue, Jun 28, 1:28 PM · Restricted Project, Restricted Project
reames requested review of D128758: [RISCV] Refine known bits for READ_VLENB.
Tue, Jun 28, 1:26 PM · Restricted Project, Restricted Project
reames added a comment to D128741: [RISCV] Fold (add X, [-4096, -2049]) or (add X, [2048,4096]) into load/store address during isel..

Code is fine, but no tests?

Tue, Jun 28, 11:03 AM · Restricted Project, Restricted Project
reames committed rGc755bf658f7d: [RISCV] Add test coverage for high known bits for vscale (authored by reames).
[RISCV] Add test coverage for high known bits for vscale
Tue, Jun 28, 10:04 AM · Restricted Project, Restricted Project
reames added a comment to D128719: [IR] Remove support for insertvalue constant expression.

Can I ask you to separate the various construction to fold changes into a separate NFC? I'd like to see that land and bake for a couple days before the removal itself goes in.

Tue, Jun 28, 8:45 AM · Restricted Project, Restricted Project

Mon, Jun 27

reames committed rG20dd3297b1c0: [LV] Allow scalable vectorization with vscale = 1 (authored by reames).
[LV] Allow scalable vectorization with vscale = 1
Mon, Jun 27, 1:39 PM · Restricted Project, Restricted Project
reames closed D128542: [LV] Allow scalable vectorization with vscale = 1.
Mon, Jun 27, 1:39 PM · Restricted Project, Restricted Project
reames committed rG0533b6e2f605: [RISCV] Remove a use of getMinVLen in favor of getRealMinVLen (authored by reames).
[RISCV] Remove a use of getMinVLen in favor of getRealMinVLen
Mon, Jun 27, 12:53 PM · Restricted Project, Restricted Project
reames committed rGaadc9d26a361: [RISCV] Cost model for scalable reductions (authored by reames).
[RISCV] Cost model for scalable reductions
Mon, Jun 27, 12:45 PM · Restricted Project, Restricted Project
reames closed D127447: [RISCV] Cost model for scalable reductions.
Mon, Jun 27, 12:44 PM · Restricted Project, Restricted Project
reames added a comment to D128627: [RISCV] Support VECTOR_REVERSE mask operation..

LGTM, but can you make sure to post a follow up patch to update cost model please?

Mon, Jun 27, 11:39 AM · Restricted Project, Restricted Project
reames accepted D125795: [IR] Remove support for extractvalue constant expression.

LGTM

Mon, Jun 27, 11:37 AM · Restricted Project, Restricted Project, Restricted Project

Sat, Jun 25

reames committed rGb61235739f64: Fix build after ab736a27 (authored by reames).
Fix build after ab736a27
Sat, Jun 25, 12:10 PM · Restricted Project, Restricted Project
reames committed rG9803b0d1e7b3: [RISCV] Implement getVScaleForTuning and thus prefer scalable vectorization… (authored by reames).
[RISCV] Implement getVScaleForTuning and thus prefer scalable vectorization…
Sat, Jun 25, 11:26 AM · Restricted Project, Restricted Project
reames closed D128547: [RISCV] Implement getVScaleForTuning and thus prefer scalable vectorization when enabled.
Sat, Jun 25, 11:25 AM · Restricted Project, Restricted Project
reames committed rGab736a2750a9: [BasicTTI] Account for vector of pointers in getMemoryOpCost (authored by reames).
[BasicTTI] Account for vector of pointers in getMemoryOpCost
Sat, Jun 25, 11:17 AM · Restricted Project, Restricted Project
reames closed D128228: [BasicTTI] Account for vector of pointers in getMemoryOpCost.
Sat, Jun 25, 11:17 AM · Restricted Project, Restricted Project
reames committed rG767ba58f808d: [RISCV] Make getMinRVVVectorSizeInBits and getMaxRVVVectorSizeInBits protected… (authored by reames).
[RISCV] Make getMinRVVVectorSizeInBits and getMaxRVVVectorSizeInBits protected…
Sat, Jun 25, 11:12 AM · Restricted Project, Restricted Project

Fri, Jun 24

reames committed rG4710e789741f: [RISCV] Implement RISCVTTIImpl::getMaxVScale correctly (authored by reames).
[RISCV] Implement RISCVTTIImpl::getMaxVScale correctly
Fri, Jun 24, 4:52 PM · Restricted Project, Restricted Project
reames closed D128538: [RISCV] Implement RISCVTTIImpl::getMaxVScale correctly.
Fri, Jun 24, 4:52 PM · Restricted Project, Restricted Project
reames committed rGa0443dd47c84: [RISCV] Simplify 16 bit index handling in lowerVECTOR_REVERSE [nfc] (authored by reames).
[RISCV] Simplify 16 bit index handling in lowerVECTOR_REVERSE [nfc]
Fri, Jun 24, 1:17 PM · Restricted Project, Restricted Project
reames committed rGf1e1c3ce7725: [RISCV] Replace two calls to getMinRVVVectorSizeInBits in fixed length lowering… (authored by reames).
[RISCV] Replace two calls to getMinRVVVectorSizeInBits in fixed length lowering…
Fri, Jun 24, 1:01 PM · Restricted Project, Restricted Project
reames committed rGf1b1bcdbd4b4: [RISCV] Replace two calls to getMinRVVVectorSizeInBits with getRealMinVLen [nfc] (authored by reames).
[RISCV] Replace two calls to getMinRVVVectorSizeInBits with getRealMinVLen [nfc]
Fri, Jun 24, 12:08 PM · Restricted Project, Restricted Project
reames requested review of D128547: [RISCV] Implement getVScaleForTuning and thus prefer scalable vectorization when enabled.
Fri, Jun 24, 11:54 AM · Restricted Project, Restricted Project
reames committed rGae8fac6f98dd: [LV][RISCV] Add coverage showing scalable codegen when etype != ELEN (authored by reames).
[LV][RISCV] Add coverage showing scalable codegen when etype != ELEN
Fri, Jun 24, 11:39 AM · Restricted Project, Restricted Project
reames updated the summary of D128542: [LV] Allow scalable vectorization with vscale = 1.
Fri, Jun 24, 11:31 AM · Restricted Project, Restricted Project
reames requested review of D128542: [LV] Allow scalable vectorization with vscale = 1.
Fri, Jun 24, 11:27 AM · Restricted Project, Restricted Project
reames requested review of D128538: [RISCV] Implement RISCVTTIImpl::getMaxVScale correctly.
Fri, Jun 24, 9:14 AM · Restricted Project, Restricted Project
Anthonysidess awarded rG056d63938a6f: [RISCV] Split a vectorizer test runline so that upcoming changes in defaults… a 100 token.
Fri, Jun 24, 8:58 AM · Restricted Project
reames committed rG056d63938a6f: [RISCV] Split a vectorizer test runline so that upcoming changes in defaults… (authored by reames).
[RISCV] Split a vectorizer test runline so that upcoming changes in defaults…
Fri, Jun 24, 8:48 AM · Restricted Project
reames committed rGadbe718675b3: [RISCV] Modify a test line so it exercises the intended configuration once we… (authored by reames).
[RISCV] Modify a test line so it exercises the intended configuration once we…
Fri, Jun 24, 8:48 AM · Restricted Project, Restricted Project

Thu, Jun 23

reames committed rG7bfad7b9d8f9: [RISCV] Replace two calls to getMinRVVVectorSizeInBits with… (authored by reames).
[RISCV] Replace two calls to getMinRVVVectorSizeInBits with…
Thu, Jun 23, 4:00 PM · Restricted Project, Restricted Project
reames committed rG46ea4b5ea10f: [LV] Avoid a crash when costing a uniform store which doesn't correspond to a… (authored by reames).
[LV] Avoid a crash when costing a uniform store which doesn't correspond to a…
Thu, Jun 23, 12:42 PM · Restricted Project, Restricted Project
reames added a comment to D128310: [RISCV] Move vfma_vl+fneg_vl matching to DAG combine..

Direction looks reasonable, a couple comments.

Thu, Jun 23, 11:21 AM · Restricted Project, Restricted Project
reames committed rG1cc97922811a: [RISCV] Fix a crash in InsertVSETVLI where we hadn't properly guarded for a… (authored by reames).
[RISCV] Fix a crash in InsertVSETVLI where we hadn't properly guarded for a…
Thu, Jun 23, 10:25 AM · Restricted Project, Restricted Project
reames added a comment to D128286: [RISCV] Disable <vscale x 1 x *> types with Zve32x or Zve32f..

Warning, I may be miss understanding the problem you're solving here, but...

The case you mention appear to be specific to when VLEN=32 right? If so, a cleaner way of phrasing the illegal cases would seem to be to compute the effective vector length after LMUL (i.e. VLEN/8 for mf8), and then disallow any case where the implied vector length is shorter than a single element of the element type.

The problem is still valid for VLEN>=64 ELEN=32. While you are correct that a SEW=8 LMUL=1/8 would fit in a register for that config, there is no requirement in the spec for hardware to support it. The spec says that that the smallest fractional LMUL is SEWMIN/ELEN. Which would be 8/32 or 1/4 in this config. The spec goes on to say "For a given supported fractional LMUL setting, implementations must support SEW settings between SEWMIN and LMUL * ELEN, inclusive." So if ELEN is 32, LMUL=1/4 is only required to support SEW=8 and LMUL=1/2 is only required to support SEW=8 or 16.

I think I can still phrase that in the "does this fit" manner. I just need to introduce a MINVLEN (which is simply ELEN), and ask whether an implied vector length for a given LMUL contains a least one element.

Thu, Jun 23, 9:23 AM · Restricted Project, Restricted Project
reames committed rG0c1326748f66: [BasicTTI] Avoid crash when costing scalable select expansion (authored by reames).
[BasicTTI] Avoid crash when costing scalable select expansion
Thu, Jun 23, 9:15 AM · Restricted Project, Restricted Project
reames added a comment to D128286: [RISCV] Disable <vscale x 1 x *> types with Zve32x or Zve32f..

Warning, I may be miss understanding the problem you're solving here, but...

Thu, Jun 23, 8:43 AM · Restricted Project, Restricted Project
reames accepted D128444: [BasicAA] Handle passthru calls in isEscapeSource().

LGTM, good find.

Thu, Jun 23, 8:33 AM · Restricted Project, Restricted Project
reames added inline comments to D128393: [RISCV] Add macrofusion infrastructure and one example usage..
Thu, Jun 23, 8:10 AM · Restricted Project, Restricted Project

Wed, Jun 22

reames accepted D128393: [RISCV] Add macrofusion infrastructure and one example usage..

LGTM w/one required change.

Wed, Jun 22, 6:17 PM · Restricted Project, Restricted Project
reames added a comment to D128322: [GuardWidening] Use logical and in widenCondCommon as it stated in doc.

Hm, this adds quite a bit of complexity.

Wed, Jun 22, 7:59 AM · Restricted Project, Restricted Project

Tue, Jun 21

reames updated the diff for D128228: [BasicTTI] Account for vector of pointers in getMemoryOpCost.

Rebase over newly added tests - after thinking about this a bit more, I realized this was exerciseable through CostModel.

Tue, Jun 21, 2:50 PM · Restricted Project, Restricted Project
reames committed rG235ba26dd67c: [RISCV] Add cost model test coverage for loads and stores (both fixed and… (authored by reames).
[RISCV] Add cost model test coverage for loads and stores (both fixed and…
Tue, Jun 21, 2:43 PM · Restricted Project, Restricted Project
reames committed rG8ae0664282dc: LoopVect, tests] Add some basic coverage for scalable costing of scatter/gather… (authored by reames).
LoopVect, tests] Add some basic coverage for scalable costing of scatter/gather…
Tue, Jun 21, 1:58 PM · Restricted Project, Restricted Project
reames updated the summary of D128228: [BasicTTI] Account for vector of pointers in getMemoryOpCost.
Tue, Jun 21, 1:54 PM · Restricted Project, Restricted Project
reames added a comment to rGbbf3fd4af165: [BasicTTI] Return Invalid for scalable vectors reaching getScalarizationOverhead.

For the record, the minimal test effected by this was:
void e(int *argv[], int *p) {

for (int i = 0; i < 1024; i++)
  argv[i] = p;

}

Tue, Jun 21, 1:52 PM · Restricted Project, Restricted Project
reames committed rG2cf320d41ed7: [LoopVect, tests] Add some basic coverage for scalable costing on RISCV (authored by reames).
[LoopVect, tests] Add some basic coverage for scalable costing on RISCV
Tue, Jun 21, 1:36 PM · Restricted Project, Restricted Project
reames added a comment to D126461: [RISCV] Extract and store new vl of vleff/vlsegff iff new_vl output pointer isn't null.

Ok, with the revised description, let me start anew in my response.

Tue, Jun 21, 12:07 PM · Restricted Project, Restricted Project
reames added a comment to D126461: [RISCV] Extract and store new vl of vleff/vlsegff iff new_vl output pointer isn't null.

@craig.topper Much clearer, thank you.

Tue, Jun 21, 11:45 AM · Restricted Project, Restricted Project
reames added a comment to D128155: [GuardWidening] Use freeze to widen using possible poison value..

Logical and is select i1 %c1, i1 %c2, i1 false.

Also, IRBuilder::CreateLogicalAnd

Tue, Jun 21, 7:37 AM · Restricted Project, Restricted Project
reames added a comment to D126461: [RISCV] Extract and store new vl of vleff/vlsegff iff new_vl output pointer isn't null.

dst in the patch description is not the pointer being loaded, it's the pointer of where to store the new_vl. That is only thing being checked for null in this patch.

I agree this is a possible interpretation, but it's also inconsistent with some of the other review discussion above.

Tue, Jun 21, 7:35 AM · Restricted Project, Restricted Project

Mon, Jun 20

reames requested review of D128228: [BasicTTI] Account for vector of pointers in getMemoryOpCost.
Mon, Jun 20, 2:27 PM · Restricted Project, Restricted Project
reames committed rGbbf3fd4af165: [BasicTTI] Return Invalid for scalable vectors reaching getScalarizationOverhead (authored by reames).
[BasicTTI] Return Invalid for scalable vectors reaching getScalarizationOverhead
Mon, Jun 20, 1:53 PM · Restricted Project, Restricted Project
reames committed rG0aebd1d87586: [RISCV] Fix crash when costing scalable gather/scatter of pointer (authored by reames).
[RISCV] Fix crash when costing scalable gather/scatter of pointer
Mon, Jun 20, 12:51 PM · Restricted Project, Restricted Project
reames closed D128219: [RISCV] Fix crash when costing scalable gather/scatter of pointer.
Mon, Jun 20, 12:51 PM · Restricted Project, Restricted Project
reames requested review of D128219: [RISCV] Fix crash when costing scalable gather/scatter of pointer.
Mon, Jun 20, 12:11 PM · Restricted Project, Restricted Project
reames added a comment to D127447: [RISCV] Cost model for scalable reductions.

ping

Mon, Jun 20, 10:53 AM · Restricted Project, Restricted Project
reames planned changes to D126884: [RISCV] Hoist vsetvli with vreg operand out of loops.

Went to land this today - I've been intentionally spacing out changes since I have a lot of churn in this file - and encountered a new crash in a test failure. Need to investigate and probably re-review once cause is understood.

Mon, Jun 20, 10:43 AM · Restricted Project, Restricted Project
reames added inline comments to D127680: [BasicTTI] Allow generic handling of scalable vector fshr/fshl.
Mon, Jun 20, 10:41 AM · Restricted Project, Restricted Project
reames committed rGdb85345f2d9f: [BasicTTI] Allow generic handling of scalable vector fshr/fshl (authored by reames).
[BasicTTI] Allow generic handling of scalable vector fshr/fshl
Mon, Jun 20, 10:39 AM · Restricted Project, Restricted Project
reames closed D127680: [BasicTTI] Allow generic handling of scalable vector fshr/fshl.
Mon, Jun 20, 10:39 AM · Restricted Project, Restricted Project
reames committed rG14847098f90b: [RISCV] Delete unexercised VL=0 vsetvli compatibility logic (authored by reames).
[RISCV] Delete unexercised VL=0 vsetvli compatibility logic
Mon, Jun 20, 10:15 AM · Restricted Project, Restricted Project
reames added inline comments to D127680: [BasicTTI] Allow generic handling of scalable vector fshr/fshl.
Mon, Jun 20, 9:02 AM · Restricted Project, Restricted Project
reames requested changes to D126461: [RISCV] Extract and store new vl of vleff/vlsegff iff new_vl output pointer isn't null.

Despite the comments above, the purpose of this patch remains unclear.

Mon, Jun 20, 8:27 AM · Restricted Project, Restricted Project
reames committed rGdc562d570dfe: [RISCV] Fold prepass back into InsertVSETVLI data flow [nfc-ish] (authored by reames).
[RISCV] Fold prepass back into InsertVSETVLI data flow [nfc-ish]
Mon, Jun 20, 7:57 AM · Restricted Project, Restricted Project
reames closed D127870: [RISCV] Fold prepass back into InsertVSETVLI data flow [nfc-ish].
Mon, Jun 20, 7:56 AM · Restricted Project, Restricted Project
reames committed rG820e84e05017: [RISCV] Assert initial load/store SEW is the EEW (authored by reames).
[RISCV] Assert initial load/store SEW is the EEW
Mon, Jun 20, 7:45 AM · Restricted Project, Restricted Project
reames closed D128085: [RISCV] Assert initial load/store SEW is the EEW.
Mon, Jun 20, 7:45 AM · Restricted Project, Restricted Project

Fri, Jun 17

reames accepted D128099: [RISCV] Pre-promote v1i1/v2i1/v4i1->i1/i2/i4 bitcasts before type legalization.

LGTM

Fri, Jun 17, 3:13 PM · Restricted Project, Restricted Project
reames committed rGfb8ecca06f6a: [RISCV] Remove redundant code checking for exact VTYPE match [nfc] (authored by reames).
[RISCV] Remove redundant code checking for exact VTYPE match [nfc]
Fri, Jun 17, 12:21 PM · Restricted Project, Restricted Project
reames committed rG4d245f1bc2e2: [RISCV] Move store policy and mask reg ops into demanded handling in… (authored by reames).
[RISCV] Move store policy and mask reg ops into demanded handling in…
Fri, Jun 17, 12:13 PM · Restricted Project, Restricted Project
reames added a comment to D128023: [RISCV] Add merge operand to RISCVISD::VRGATHER*_VL nodes..

Looks reasonable to me, though the TD changes are a bit more involved than I'm comfortable signing off on just yet. Will defer to Fraser.

Fri, Jun 17, 12:05 PM · Restricted Project, Restricted Project
reames updated the diff for D128085: [RISCV] Assert initial load/store SEW is the EEW.

Wrap code in macro to avoid release build warning.

Fri, Jun 17, 11:56 AM · Restricted Project, Restricted Project
reames requested review of D128085: [RISCV] Assert initial load/store SEW is the EEW.
Fri, Jun 17, 11:52 AM · Restricted Project, Restricted Project
reames committed rGb595cddea7fe: [riscv] Extract isMaskRegOp helper [nfc] (authored by reames).
[riscv] Extract isMaskRegOp helper [nfc]
Fri, Jun 17, 10:41 AM · Restricted Project, Restricted Project
reames committed rGe1f1407beb84: [RISCV] Delete dead elideCopy code in InsertVSETVLI [nfc] (authored by reames).
[RISCV] Delete dead elideCopy code in InsertVSETVLI [nfc]
Fri, Jun 17, 9:58 AM · Restricted Project, Restricted Project
reames closed D127996: [RISCV] Delete dead elideCopy code in InsertVSETVLI [nfc].
Fri, Jun 17, 9:58 AM · Restricted Project, Restricted Project
reames committed rG755c84c62cda: [RISCV] Avoid changing etype for splat of 0 or -1 (authored by reames).
[RISCV] Avoid changing etype for splat of 0 or -1
Fri, Jun 17, 8:10 AM · Restricted Project, Restricted Project
reames closed D128006: [RISCV] Avoid changing etype for splat of 0 or -1.
Fri, Jun 17, 8:10 AM · Restricted Project, Restricted Project
reames added inline comments to D128006: [RISCV] Avoid changing etype for splat of 0 or -1.
Fri, Jun 17, 8:07 AM · Restricted Project, Restricted Project

Thu, Jun 16

reames added a comment to D128016: Recommit "[RISCV] Enable subregister liveness tracking for RVV.".

LGTM w/minor optional comments.

Thu, Jun 16, 3:56 PM · Restricted Project, Restricted Project
reames updated the summary of D127447: [RISCV] Cost model for scalable reductions.
Thu, Jun 16, 3:55 PM · Restricted Project, Restricted Project
reames updated the diff for D127447: [RISCV] Cost model for scalable reductions.

Rework with the max-vl based approach we used in scatter/gather.

Thu, Jun 16, 3:52 PM · Restricted Project, Restricted Project