[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