Re: Fix for the issue 12176

2017-09-29 Thread Oscar Forner Martinez via cfe-commits
On 28/09/17 21:28, Jonathan Roelofs wrote: +silvas On 9/28/17 2:19 PM, Oscar Forner Martinez via cfe-commits wrote: Hi, Please find attached a diff to fix the issue 12176. Link for the lazy: llvm.org/PR12176 Thanks ;) Let me know if there is anything any improvements you can think

Re: Fix for the issue 12176

2017-09-28 Thread Oscar Forner Martinez via cfe-commits
Hi Jonathan, Thanks for the input. I will amend it and update it. I created an entry in Phabricator too (https://reviews.llvm.org/D38382). Best regards, Oscar On 28/09/17 21:28, Jonathan Roelofs wrote: +silvas On 9/28/17 2:19 PM, Oscar Forner Martinez via cfe-commits wrote: Hi, Please

Re: Fix for the issue 12176

2017-09-28 Thread Jonathan Roelofs via cfe-commits
+silvas On 9/28/17 2:19 PM, Oscar Forner Martinez via cfe-commits wrote: Hi, Please find attached a diff to fix the issue 12176. Link for the lazy: llvm.org/PR12176 Let me know if there is anything any improvements you can think of. Best regards, Oscar