[all-commits] [llvm/llvm-project] 8fe71e: [support] Don't require VFS in `SourceMgr` for loa...

Jan Svoboda via All-commits all-commits at lists.llvm.org
Fri Oct 17 08:12:56 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8fe71e0bdfb9102c607001289010698e51e38711
      https://github.com/llvm/llvm-project/commit/8fe71e0bdfb9102c607001289010698e51e38711
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2025-10-17 (Fri, 17 Oct 2025)

  Changed paths:
    M llvm/lib/Support/SourceMgr.cpp

  Log Message:
  -----------
  [support] Don't require VFS in `SourceMgr` for loading includes (#163862)

This commit more gracefully handles situations where `SourceMgr` isn't
initialized with a VFS and tries to resolve an include. That's what
happens with the test case `clang -flto=thin -c test.c -o /dev/null`
where test.c contains `asm(" .incbin \"foo.i\" \n");`. Propagating the
actual VFS all the way is very difficult.

This is a follow-up to #162903.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list