Closed Quuxplusone closed 6 years ago
Bugzilla Link | PR36974 |
Status | RESOLVED FIXED |
Importance | P normal |
Reported by | Matthias Krüger (matthias.krueger@famsik.de) |
Reported on | 2018-04-02 02:08:44 -0700 |
Last modified on | 2018-04-04 01:07:43 -0700 |
Version | trunk |
Hardware | PC Linux |
CC | efriedma@quicinc.com, llvm-bugs@lists.llvm.org, serguei.katkov@azul.com |
Fixed by commit(s) | |
Attachments | |
Blocks | |
Blocked by | |
See also |
Yes, dominates() is not a strict weak ordering. The call to std::max_element should be using properlyDominates().
Eli, thank you for root cause.
Interesting that in real example you should get any problem due to LoopsUsed is a set and cannot contain two identical elements. However I will update a code to fix this bug.
(In reply to Serguei Katkov from comment #3)
> Interesting that in real example you should get any problem due to LoopsUsed
> is a set and cannot contain two identical elements. However I will update a
> code to fix this bug.
you should *not* get
Landed as rL329051. Please verify.
Thank you, fix confirmed.