[clang-tools-extra] issue-63565: community requested small QoL fix for more configurabili… (PR #108005)
    via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Sep 18 07:02:02 PDT 2024
    
    
  
================
@@ -6,6 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 
+#include "../Config.h"
----------------
MK-Alias wrote:
>This can be included without the ../
Yes, but it of-course shouldn't!  Confirm the [standard](https://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf#page=182) it should be either: `#include "../Config.h"` or `#include <Config.h>` (and also it should be `Config.hpp`)
But because this is used in the whole codebase, if changed it to `#include "Config.h"`
https://github.com/llvm/llvm-project/pull/108005
    
    
More information about the cfe-commits
mailing list