modimo added a comment.

In D124563#3478781 <https://reviews.llvm.org/D124563#3478781>, @aaronpuchert 
wrote:

> In D124563#3478653 <https://reviews.llvm.org/D124563#3478653>, @modimo wrote:
>
>> I think the way to go is to revert ac5f7be6a868 
>> <https://reviews.llvm.org/rGac5f7be6a8688955a282becf00eebc542238a86b> then 
>> land everything as a single stack to prevent this issue.
>
> Doesn't change anything about existing commits though. There is no way to fix 
> that now I'm afraid. We can only fix it for future commits.

Yeah unfortunately. On Linux since it has the timestamp check landing this in 
one piece will be clean but on Windows the existing commits will continue to 
have this issue.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124563/new/

https://reviews.llvm.org/D124563

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to