User Details
- User Since
- Jul 31 2019, 9:48 AM (183 w, 5 d)
Nov 21 2022
Nov 14 2022
@bcahoon attached a test case
Nov 1 2022
This patch is causing issue mentioned in https://discourse.llvm.org/t/sema-section-type-conflict/66000
Oct 20 2022
Hi @bcahoon, i am yet to come across a case in Hexagon where in a linear list of instruction, a use occurs before def in orderDependence(). I assume, that's never a case for Hexagon happening yet. As pointed out by @dpenry, orderDependence() is not robust for other targets to make sure that def-use are in proper order, because of which we are trying to avoid same cycle scheduling by having check in isValidSchedule() function. But since, this is not the case with Hexagon target, this patch enables same cycle def-use scheduling for only Hexagon with a flag. Let us know your thoughts over it.
Oct 18 2022
I agree that orderDependence() need to be more robust. However, we can't penalize pipeliner scheduling where def-use are in same cycle. I may add a TODO as of now and keep the flag. Once orderDependence() is fixed, we may remove the flag. Does it sound good?
Oct 16 2022
Sep 21 2019
Sep 3 2019
Added comment as suggested.
Gentle Ping !! I would be grateful if someone can review the updated patch.
Aug 20 2019
Gentle Ping!!
Aug 2 2019
Updated with 1 more test case.