Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM! This is an NFC change that doesn't require additional testing because CurEnd is only used in an assertion, not for any of the function's semantics.
Comment Actions
Hi @Manna, I'm not sure which failure you are referring to, but there is a test failure on at least 2 build bots that seems to have been caused by your change. Can you take a look and revert if you need time to investigate?
https://lab.llvm.org/buildbot/#/builders/139/builds/48141
https://lab.llvm.org/buildbot/#/builders/216/builds/26116
Comment Actions
Thank you @dyung for reporting the buildbot failure. I have reverted my patch. I will investigate it.