[PATCH] D120619: [clangd] Support include-fixer inside macro arguments.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 23 10:58:37 PDT 2022


sammccall added a comment.

In D120619#3403021 <https://reviews.llvm.org/D120619#3403021>, @thakis wrote:

> This breaks tests on windows: http://45.33.8.238/win/54910/step_9.txt
>
> Please take a look and revert for now if it takes a while to fix.

Thanks, forced the test to pass in 31dc248ffc3cbf9ea6d11819b6b7b731b414b352 <https://reviews.llvm.org/rG31dc248ffc3cbf9ea6d11819b6b7b731b414b352>.

The actual feature doesn't work on windows, and I'm not sure why (other than it's something under -fms-compatibility)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120619



More information about the cfe-commits mailing list