- User Since
- Oct 23 2013, 8:22 AM (399 w, 5 d)
Mon, May 31
Revert to previous revision Diff 347356 and add empty module flag could be empty in test.
Sat, May 29
Good catch! LGTM.
Thu, May 27
address @luismarques's comments.
Mon, May 24
- Handle an empty module flag.
So sorry I forgot to run the clang regresion tests before.
- Add empty module flag test and combine all tests into one. It is simiar to compress.ll
I think this patch is similar to D73339 but it fixes obj file attribute, it is not going to fix different target feature in one complication unit problem.
Sun, May 23
clang or clang+llc your test with my patch the arch attribute is still rv64i2p0, does it break your use cases?
update test to show asm has F instructions.
May 22 2021
I got the similar IR during LTO because the clang driver would not pass the -mattr option to lto code generator.
May 21 2021
May 17 2021
address Craig's comments, thanks!
addres https://reviews.llvm.org/D71387#2762120, compute a default ABI for comparison rather than report an error when missing -mabi
May 16 2021
- address @jrtc27' comment
- change report_fatal_error as errs() because report_fatal_error will ask users to submit a bug report, it does not make sense.
- combine two error message as one.
Pass -target-abi option into LTO codegenerator base on D102582 patch.
May 9 2021
Good catch, LGTM!
May 7 2021
Good catch! LGTM!
May 5 2021
May 3 2021
Hi, I would like to add ilp32e ABI support in llvm
Is there anyone working on this?
It seem the one thing missed is ilp32e ABI should disallow D ISA extension.
Is there anything else?
May 2 2021
LGTM. Thanks for improvement!
Apr 28 2021
Apr 27 2021
Thanks for clarification, LGTM.
Apr 26 2021
Select the immediate during isel does make sense to me, but unfortunately there are some cases have a slower result.
Do you know is there any cases which have better instruction order and reduce register spilling when apply the new scheme?
I'm just afraid of the new one would be always generate the slower instruction order.
Apr 25 2021
Apr 21 2021
Apr 20 2021
Apr 19 2021
Apr 18 2021
LGTM. Please remove the ASM check in upstream patches.
Look good to me.
BTW, we also need to update the document later.
Apr 17 2021
I think you also need to handle PermuteOperands field?
PermuteOperands performs the order permutation for non-masked and masked intrinsics when the operand order is different to builtins.
Apr 16 2021
Address the Craig's comments.
Apr 15 2021
I think using this proxy header for testing seems good.
The fact that we're running the optimization pipeline in these tests might be to blame. Might also be that riscv_vector.h is currently about 71000 lines which probably isn't quick to parse.
Apr 14 2021
Apr 13 2021
Apr 12 2021
address Jim's comment.
Apr 11 2021
Good catch! LGTM.
Apr 9 2021
address Craig's comment.
Apr 8 2021