[PATCH] D36963: Use sorting instead of a lot of stable_partitions for `ICF::segregate` function

Alex Telishev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 07:47:18 PDT 2017


alex.telishev updated this revision to Diff 112560.
alex.telishev added a comment.

Fix style.
I have a question - why opt:icf is the default option? msvc do it only for release builds and I think they're right about it - opt:icf make debugging very hard as different functions that have the same body will be merged. I can send a patch with a fix for it.


https://reviews.llvm.org/D36963

Files:
  lld/COFF/ICF.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36963.112560.patch
Type: text/x-patch
Size: 6436 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170824/c08378b9/attachment.bin>


More information about the llvm-commits mailing list