- User Since
- Jun 17 2015, 7:07 AM (126 w, 5 d)
Sun, Nov 19
Remove duplicated empty lines.
Looks great (whatever you decide to to with the inline comment)
Thu, Nov 16
Tue, Nov 14
I cannot apply this patch, arcanist always tries to apply it on git sha1 85a1620 (current trunk without D39979).
LGTM, only nitpicks. Thanks for your work.
Thu, Nov 9
Mon, Nov 6
LGTM, fails for me too. Probably caused by change in clang-format: rL317325
Tue, Oct 31
Abandon because the alternative was committed
Mon, Oct 30
- Address Siddharth's remarks
- Fix one more TODO
- Introduce isNormalizable to resolve TODO
- Remove unused function attributes in test-case
- Add example to NormalizedPHI doxygen
Fri, Oct 27
For me this is OK. We cannot reliably test undefined behavior. Tools like TypeSanitizer exist for this purpose.
Thu, Oct 26
Add selfrefphi test case
Tue, Oct 24
Oct 20 2017
Could you check whether this works for you:
Oct 17 2017
Thanks for improving our documentation!
Oct 6 2017
Please add at least one test case where a basic block is successfully split up and check that there are two statements now.
Oct 5 2017
- Address comments
- Fix return instead of continue bug
Oct 4 2017
- Add unit tests
- Handle scalar dependencies due to incoming values
- Add comments
Oct 2 2017
Oct 1 2017
Sep 29 2017
Sep 28 2017
Sep 27 2017
For reference: the alternative is D38308.
Sep 25 2017
Sep 22 2017
Sep 21 2017
Sep 20 2017
Sep 19 2017
Thanks, looks good. I'll commit after D37337.
Please add at least one test case for the new feature (splitting at stores).
Sep 18 2017
Sorry for my absence for a while, a paper submission deadline came in the way.
Sep 9 2017
Sep 7 2017
Add ExpensiveCombines argument.
Sep 6 2017
Sep 3 2017
I assume that not everyone subscribed to llvm-commits knows that "ForwardOpTree"/"ForwardOp" is a component of Polly. The tag "[Polly]" might still be useful.