Displaying 2 results from an estimated 2 matches similar to: "[LLVMdev] Question about DAGCombiner::MatchRotate function"
2013 Oct 03
0
[LLVMdev] Question about DAGCombiner::MatchRotate function
Hi JinGu,
It's normally best to send patches to the llvm-commits list. That's
where most of the reviews happen.
> The problem is that code does not check this with new specific type "LArgVT"
> and "RArgVT" while processing above patterns. How do you think about this? I
> think checking code is needed.
I agree.
> I have attached simple patch to fix it.
2007 Mar 30
1
[LLVMdev] Cleanups in ROTL/ROTR DAG combiner code
The attached patch contains:
- Cleanups in the DAGCombiner.cpp ROTL/ROTR combine code, primarily
helping me to fix 80col violations (benefiting the code as a whole).
- Detect sign/zext/any-extended versions of ROTL/ROTR patterns.
- Allow custom lowering for ROTL/ROTR (needed in the CellSPU's case
for 8-bit rotates, when only 16-bit and 32-bit rotates are actually
implemented in the