tbaeder added a comment.

That code has changed quite a bit since I've worked on it.

The only problem I could see is that passing `-flto=thin -flto` and choosing 
thin LTO kinda makes sense if you interpret `-flto` as just "use LTO". The 
`-flto=thin` is more specific, so it could make sense to pick that. But that's 
just something I wonder while looking at it. Not sure if we process other 
options like this by just choosing the last one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108881

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

Reply via email to