[PATCH] D33621: Fix for -wrap linker option and LTO, PR 33145

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 11:49:55 PDT 2017

Previous message: [PATCH] D33621: Fix for -wrap linker option and LTO, PR 33145
Next message: [PATCH] D33892: Align definition of DW_OP_plus with DWARF spec [1/3]
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

davide added a comment.

There were some comments pending by Rui. You addressed them, but you still need to wait for an explicit `LGTM` before committing.
I don't think it's worth reverting this patch, but this is not okay and you should consider it next time.


Repository:
  rL LLVM

https://reviews.llvm.org/D33621

Previous message: [PATCH] D33621: Fix for -wrap linker option and LTO, PR 33145
Next message: [PATCH] D33892: Align definition of DW_OP_plus with DWARF spec [1/3]
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list