balazske added a comment.

The problem with small changes is that still we (at least I who writes the 
code) should know the final goal (and this can be hard if I have multiple not 
related problems to work on and everything goes forward by little pieces spread 
out in a long time). (Also the reviewer needs to know why a change is made if 
it is not obvious because it is part of a bigger to-be-added "logic".) 
Otherwise something already done can turn out to be wrong later, as is the case 
already now with these changes. My goal was to add the file handling functions 
one by one and adjust the existing code to work with the current implemented 
functions (probably something already added must be changed to adapt to the new 
conditions).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78374



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

Reply via email to