[all-commits] [llvm/llvm-project] 2c1fa7: Reland "[Lex] Fix suggested spelling of /usr/bin/....

Sam McCall via All-commits all-commits at lists.llvm.org
Mon Nov 28 01:09:40 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2c1fa734598c9470139720565fbf624a5156ec03
      https://github.com/llvm/llvm-project/commit/2c1fa734598c9470139720565fbf624a5156ec03
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2022-11-28 (Mon, 28 Nov 2022)

  Changed paths:
    M clang/lib/Lex/HeaderSearch.cpp
    M clang/unittests/Lex/HeaderSearchTest.cpp

  Log Message:
  -----------
  Reland "[Lex] Fix suggested spelling of /usr/bin/../include/foo"

This reverts commit 1dc0a1e5d220b83c1074204bd3afd54f3bac4270.

Failures were caused by unintentional conversion to native slashes by
remove_dots, so undo that: we always suggest posix slashes for includes.

This could potentially be a change in behavior on windows if people were
spelling headers with backslashes and headermaps contained backslashes,
but that's all underspecified and I don't think anyone uses headermaps
on windows.

Differential Revision: https://reviews.llvm.org/D138709




More information about the All-commits mailing list