- User Since
- Jan 30 2014, 6:27 AM (272 w, 1 d)
To follow up on my comment why this is NVPTX specific:
I'm not really an expert in InstCombine but it looks good to me. Someone else should probably take a look as well. I added some minor comments though.
Thu, Apr 18
Improved and added various comments & documentation
Wed, Apr 17
Initial high-level file comment, more to come!
Addressed most comments, some I replied back to. Let me know if I should change those things as well.
Changes according to most of Chandler's comments
I inlined a question. Also, did you run the tests with assertions enabled?
Tue, Apr 16
I hope D60076 doesn't have the same problem ;)
Mon, Apr 15
Fri, Apr 12
Thu, Apr 11
Some initial comments from my side.
Sun, Apr 7
Fri, Apr 5
Wed, Apr 3
Tue, Apr 2
I might have missed some earlier discussion but how does this solve the problem? Also, I assume the problem is still PR41047, correct?
Mon, Apr 1
Improve bug-exposing test cases (see http://llvm.org/PR41328)
Minor adjustments wrt later patches
Sun, Mar 31
What's the status here? How can we make this go in?
Sat, Mar 30
Simplify the interface & make it impossible to break invariances
Add bug-exposing test case
Fri, Mar 29
Closed a side-channel through "integers"
Minor adjustment to prior patches
Introduce later parts and minor fixes
Add more tests and fix misleading comments